Github user QiangCai commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/3001#discussion_r245916004
--- Diff:
integration/presto/src/main/java/org/apache/carbondata/presto/readers/BooleanStreamReader.java
---
@@ -86,4 +86,15 @@ public BooleanStreamReader(int batchSize, DataType
dataType, Dictionary dictiona
builder = type.createBlockBuilder(null, batchSize);
}
+ @Override public void putObject(int rowId, Object value) {
--- End diff --
It is only required by stream reader. But I don't want to create many new
adaptive objects, it will become complex.
---