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

    https://github.com/apache/carbondata/pull/2347#discussion_r191399826
  
    --- Diff: 
processing/src/main/java/org/apache/carbondata/processing/loading/steps/InputProcessorStepWithNoConverterImpl.java
 ---
    @@ -313,7 +315,22 @@ private CarbonRowBatch getBatch() {
                   throw new CarbonDataLoadingException("Loading Exception", e);
                 }
               } else {
    -            newData[i] = data[orderOfData[i]];
    +            DataType dataType = dataFields[i].getColumn().getDataType();
    +            if (dataType == DataTypes.DATE && data[orderOfData[i]] 
instanceof Long) {
    +              DirectDictionaryGenerator directDictionaryGenerator =
    --- End diff --
    
    why everytime a new directDictionaryGenerator object is needed? It can also 
be a member variable of InputProcessorStepWithNoConverterImpl and initialize 
only once. 


---

Reply via email to