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

    https://github.com/apache/carbondata/pull/2209#discussion_r184929215
  
    --- Diff: 
core/src/main/java/org/apache/carbondata/core/scan/complextypes/PrimitiveQueryType.java
 ---
    @@ -53,6 +55,21 @@ public PrimitiveQueryType(String name, String 
parentname, int blockIndex,
         this.name = name;
         this.parentname = parentname;
         this.isDirectDictionary = isDirectDictionary;
    +    this.isDictionary = true;
    +  }
    +
    +
    +  public PrimitiveQueryType(String name, String parentname, int blockIndex,
    --- End diff --
    
    Moved into a single constructor. But we cannot check is the column is a 
dictionary or nodictionary through the existing variable dictionary as it is 
always going to get filled up by default. 


---

Reply via email to