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

    https://github.com/apache/carbondata/pull/2875#discussion_r230629760
  
    --- Diff: 
core/src/main/java/org/apache/carbondata/core/constants/CarbonCommonConstants.java
 ---
    @@ -117,77 +126,106 @@ private CarbonCommonConstants() {
        * CARBON_TIMESTAMP
        */
       public static final String CARBON_TIMESTAMP_MILLIS = "dd-MM-yyyy 
HH:mm:ss:SSS";
    +
       /**
    -   * compressor for writing/reading carbondata file
    +   * Property for specifying the format of DATE data type column.
    +   * e.g. yyyy/MM/dd , or using default value
        */
    +  @CarbonProperty
    +  public static final String CARBON_DATE_FORMAT = "carbon.date.format";
    +
    +  /**
    +   * default value
    +   */
    +  public static final String CARBON_DATE_DEFAULT_FORMAT = "yyyy-MM-dd";
    +
    +  /**
    +   * compressor for writing/reading CarbonData file
    +   */
    +  @CarbonProperty
       public static final String COMPRESSOR = "carbon.column.compressor";
     
       /**
        * default compressor is snappy
        */
       public static final String DEFAULT_COMPRESSOR = "snappy";
    +
       /**
    -   * ZOOKEEPER_ENABLE_LOCK if this is set to true then zookeeper will be 
used to handle locking
    +   * ZOOKEEPER_ENABLE_LOCK if this is set to true then zookeeper
    +   * will be used to handle locking
        * mechanism of carbon
        */
       @CarbonProperty
       public static final String LOCK_TYPE = "carbon.lock.type";
     
    +  /**
    +   * ZOOKEEPER_ENABLE_DEFAULT the default value for zookeeper will be true 
for carbon
    +   */
    +  public static final String LOCK_TYPE_DEFAULT = "LOCALLOCK";
    +
       /**
        * Specifies the path where the lock files have to be created.
        * By default, lock files are created in table path.
        */
       @CarbonProperty
       public static final String LOCK_PATH = "carbon.lock.path";
     
    -  /**
    -   * ZOOKEEPER_ENABLE_DEFAULT the default value for zookeeper will be true 
for carbon
    -   */
    -  public static final String LOCK_TYPE_DEFAULT = "LOCALLOCK";
    +  public static final String LOCK_PATH_DEFAULT = "";
     
       /**
        * ZOOKEEPER_LOCATION this is the location in zookeeper file system 
where locks are created.
        * mechanism of carbon
        */
       public static final String ZOOKEEPER_LOCATION = "/CarbonLocks";
    +
       /**
    -   * xxhash algorithm property for hashmap
    +   * xxhash algorithm property for hashMap
        */
    +  @CarbonProperty
       public static final String ENABLE_XXHASH = "carbon.enableXXHash";
     
       /**
    -   * xxhash algorithm property for hashmap. Default value false
    +   * xxhash algorithm property for hashMap Default value false
        */
       public static final String ENABLE_XXHASH_DEFAULT = "true";
    +
       /**
        * System property to enable or disable local dictionary generation
        */
    +  @CarbonProperty
       public static final String LOCAL_DICTIONARY_SYSTEM_ENABLE = 
"carbon.local.dictionary.enable";
     
       /**
        * System property to enable or disable decoder based local dictionary 
fallback
        */
    +  @CarbonProperty
       public static final String LOCAL_DICTIONARY_DECODER_BASED_FALLBACK =
           "carbon.local.dictionary.decoder.fallback";
     
       /**
        * System property to enable or disable decoder based local dictionary 
fallback default value
        */
       public static final String 
LOCAL_DICTIONARY_DECODER_BASED_FALLBACK_DEFAULT = "true";
    +
       /**
        * zookeeper url key
        */
    +  @CarbonProperty
       public static final String ZOOKEEPER_URL = "spark.deploy.zookeeper.url";
    +
       /**
        * carbon data file version property
        */
    +  @CarbonProperty
       public static final String CARBON_DATA_FILE_VERSION = 
"carbon.data.file.version";
    +
       @CarbonProperty
       public static final String SPARK_SCHEMA_STRING_LENGTH_THRESHOLD =
           "spark.sql.sources.schemaStringLengthThreshold";
     
       public static final int SPARK_SCHEMA_STRING_LENGTH_THRESHOLD_DEFAULT = 
4000;
     
    +  //TODO: add annotation and whether change to dynamic
    --- End diff --
    
    ok, done


---

Reply via email to