[jira] [Commented] (CARBONDATA-297) 2. Add interfaces for data loading.

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[jira] [Commented] (CARBONDATA-297) 2. Add interfaces for data loading.

Akash R Nilugal (Jira)

    [ https://issues.apache.org/jira/browse/CARBONDATA-297?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15569063#comment-15569063 ]

ASF GitHub Bot commented on CARBONDATA-297:
-------------------------------------------

Github user jackylk commented on a diff in the pull request:

    https://github.com/apache/incubator-carbondata/pull/229#discussion_r83030022
 
    --- Diff: processing/src/main/java/org/apache/carbondata/processing/newflow/CarbonDataLoadConfiguration.java ---
    @@ -0,0 +1,185 @@
    +package org.apache.carbondata.processing.newflow;
    +
    +import java.util.Iterator;
    +
    +import org.apache.carbondata.core.carbon.AbsoluteTableIdentifier;
    +
    +public class CarbonDataLoadConfiguration {
    --- End diff --
   
    It seems this configuration is quite complex, I think it is because it contains configuration for all steps.
    Can we just have a simple `Map` as the configuration and let the `Step` decide what to keep in it?


> 2. Add interfaces for data loading.
> -----------------------------------
>
>                 Key: CARBONDATA-297
>                 URL: https://issues.apache.org/jira/browse/CARBONDATA-297
>             Project: CarbonData
>          Issue Type: Sub-task
>            Reporter: Ravindra Pesala
>            Assignee: Ravindra Pesala
>             Fix For: 0.2.0-incubating
>
>
> Add the major interface classes for data loading so that the following jiras can use this interfaces to implement it.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)