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

    https://github.com/apache/carbondata/pull/1581#discussion_r155438974
  
    --- Diff: 
integration/presto/src/main/java/org/apache/carbondata/presto/CarbonVectorBatch.java
 ---
    @@ -0,0 +1,112 @@
    +package org.apache.carbondata.presto;
    +
    +import java.util.Arrays;
    +import java.util.HashSet;
    +import java.util.Set;
    +
    +public class CarbonVectorBatch {
    +
    +  private static final int DEFAULT_BATCH_SIZE = 1024;
    --- End diff --
    
    The batchSize is the size of the complete batch so for every Column we have 
to set it as byteSize , it is the number of records in the batch so it has to 
be equivalent to the batch size


---

Reply via email to