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

    https://github.com/apache/incubator-carbondata/pull/265#discussion_r90399963
  
    --- Diff: 
core/src/main/java/org/apache/carbondata/core/constants/CarbonCommonConstants.java
 ---
    @@ -814,85 +810,86 @@
        * Rocord size in case of compaction.
        */
       public static final int COMPACTION_INMEMORY_RECORD_SIZE = 120000;
    -
    -  /**
    -   * If the level 2 compaction is done in minor then new compacted segment 
will end with .2
    -   */
    -  public static String LEVEL2_COMPACTION_INDEX = ".2";
    -
    -  /**
    -   * Indicates compaction
    -   */
    -  public static String COMPACTION_KEY_WORD = "COMPACTION";
    -
       /**
        * hdfs temporary directory key
        */
       public static final String HDFS_TEMP_LOCATION = "hadoop.tmp.dir";
    -
       /**
        * zookeeper url key
        */
       public static final String ZOOKEEPER_URL = "spark.deploy.zookeeper.url";
    -
       /**
        * configure the minimum blocklet size eligible for blocklet distribution
        */
       public static final String CARBON_BLOCKLETDISTRIBUTION_MIN_REQUIRED_SIZE 
=
           "carbon.blockletdistribution.min.blocklet.size";
    -
       /**
        * default blocklet size eligible for blocklet distribution
        */
       public static final int 
DEFAULT_CARBON_BLOCKLETDISTRIBUTION_MIN_REQUIRED_SIZE = 2;
    -
    +  /**
    +   * This batch size is used to send rows from load step to another step 
in batches.
    +   */
    +  public static final String DATA_LOAD_BATCH_SIZE = "DATA_LOAD_BATCH_SIZE";
    +  /**
    +   * Default size of data load batch size.
    +   */
    +  public static final String DATA_LOAD_BATCH_SIZE_DEFAULT = "1000";
    +  /**
    +   * carbon data file version property
    +   */
    +  public static final String CARBON_DATA_FILE_VERSION = 
"carbon.data.file.version";
    +  /**
    +   * current data file version
    +   */
    +  public static final short CARBON_DATA_FILE_CURRENT_VERSION = 2;
    --- End diff --
    
    change 2 to enum also


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to