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.


---

Reply via email to