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

    https://github.com/apache/carbondata/pull/2507#discussion_r202988752
  
    --- Diff: 
core/src/main/java/org/apache/carbondata/core/scan/collector/impl/DictionaryBasedVectorResultCollector.java
 ---
    @@ -109,13 +109,19 @@ void prepareDimensionAndMeasureColumnVectors() {
             allColumnInfo[queryDimensions[i].getOrdinal()] = columnVectorInfo;
           }
         }
    +    //skipping non existing measure columns in measureColumnInfo as here 
data filling to be done only on existing columns
    +    // for non existing column it is already been filled from restructure 
based collector
    +    int j = 0;
    --- End diff --
    
    @ravipesala 
    For RestructureBasedDictionaryResultCollector already handled 
implementation of filling measure of  non existing and existing  column in 
method fillmeasureData which is correct handling.


---

Reply via email to