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

    https://github.com/apache/carbondata/pull/1953#discussion_r169989543
  
    --- Diff: 
core/src/main/java/org/apache/carbondata/core/metadata/schema/BucketingInfo.java
 ---
    @@ -30,34 +30,32 @@
     /**
      * Bucketing information
      */
    -public class BucketingInfo implements Serializable, Writable {
    -
    +public class BucketingInfo implements ColumnRangeInfo, Serializable, 
Writable {
       private static final long serialVersionUID = -0L;
    -
       private List<ColumnSchema> listOfColumns;
    -
    -  private int numberOfBuckets;
    +  // number of column ranges
    --- End diff --
    
    Why is it called "column ranges"? Isn't it "value ranges"?


---

Reply via email to