[GitHub] carbondata pull request #1362: [CARBONDATA-1444]Support boolean

[jackylk]

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

    https://github.com/apache/carbondata/pull/1362#discussion_r139307234
  
    --- Diff: 
core/src/main/java/org/apache/carbondata/core/constants/CarbonCommonConstants.java
 ---
    @@ -23,6 +23,10 @@
     
     public final class CarbonCommonConstants {
       /**
    +   * Boolean size in bytes
    +   */
    +  public static final int BOOLEAN_SIZE_IN_BYTE = 1;
    --- End diff --
    
    This is not required, it should be added in DataType enum


---