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

    https://github.com/apache/carbondata/pull/1398#discussion_r143612947
  
    --- Diff: 
core/src/main/java/org/apache/carbondata/core/metadata/datatype/DataType.java 
---
    @@ -34,11 +34,13 @@
       STRUCT(10, "STRUCT", -1),
       MAP(11, "MAP", -1),
       BYTE(12, "BYTE", 1),
    -
       // internal use only, for variable length data type
       BYTE_ARRAY(13, "BYTE_ARRAY", -1),
    +
       // internal use only, for value compression from integer/long to 3 bytes 
value
    -  SHORT_INT(14, "SHORT_INT", 3);
    +  SHORT_INT(14, "SHORT_INT", 3),
    +  // Only for internal use for backward compatability
    --- End diff --
    
    mentioning it is for V1 and V2 format only


---

Reply via email to