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

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

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

    https://github.com/apache/incubator-carbondata/pull/240#discussion_r84229694
  
    --- Diff: 
processing/src/main/java/org/apache/carbondata/processing/newflow/constants/DataLoadProcessorConstants.java
 ---
    @@ -33,4 +33,8 @@
       public static final String COMPLEX_DELIMITERS = "COMPLEX_DELIMITERS";
     
       public static final String DIMENSION_LENGTHS = "DIMENSION_LENGTHS";
    +
    --- End diff --
    
    can you rename this class to a more meaningful name like `DataLoadOptions`. 
It will be exposed to user as data load options, right?


> 3. Add InputProcessorStep which should iterate recordreader and parse the 
> data as per the data type.
> ----------------------------------------------------------------------------------------------------
>
>                 Key: CARBONDATA-298
>                 URL: https://issues.apache.org/jira/browse/CARBONDATA-298
>             Project: CarbonData
>          Issue Type: Sub-task
>            Reporter: Ravindra Pesala
>             Fix For: 0.2.0-incubating
>
>
> Add InputProcessorStep which should iterate recordreader/RecordBufferedWriter 
> and parse the data as per the data types.



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

Reply via email to