Github user ajantha-bhat commented on a diff in the pull request:

    https://github.com/apache/carbondata/pull/2549#discussion_r205043434
  
    --- Diff: 
core/src/main/java/org/apache/carbondata/core/scan/collector/impl/DictionaryBasedResultCollector.java
 ---
    @@ -141,22 +141,52 @@ public 
DictionaryBasedResultCollector(BlockExecutionInfo blockExecutionInfos) {
           }
           fillMeasureData(scannedResult, row);
           if 
(scannedResult.complexParentIndexToQueryMap.toString().contains("StructQueryType"))
 {
    +        int[] isComplexColumn = new int[queryDimensions.length + 
queryMeasures.length];
    --- End diff --
    
    Please make it boolean[] , it would save space and will be meaningful to 
read.


---

Reply via email to