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

    https://github.com/apache/carbondata/pull/2227#discussion_r186276999
  
    --- Diff: 
processing/src/main/java/org/apache/carbondata/processing/loading/model/CarbonLoadModelBuilder.java
 ---
    @@ -81,9 +81,8 @@ public CarbonLoadModel build(Map<String, String> options, 
long UUID, String task
         build(options, optionsFinal, model, null);
     
     
    -    // set default values
    -    
model.setTimestampformat(CarbonCommonConstants.CARBON_TIMESTAMP_DEFAULT_FORMAT);
    -    model.setDateFormat(CarbonCommonConstants.CARBON_DATE_DEFAULT_FORMAT);
    +    // set default values other than time format
    --- End diff --
    
    This comment can be removed


---

Reply via email to