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.
---