Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2209#discussion_r184917127
--- 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,
+ org.apache.carbondata.core.metadata.datatype.DataType dataType, int
keySize,
+ Dictionary dictionary, boolean isDirectDictionary, boolean
isDictionary) {
--- End diff --
I don't think it is required to have a separate constructor, if (dictionary
== null && !isDirectDictionary) then it becomes nodictionary
---
