[ 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)