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

    https://github.com/apache/carbondata/pull/2100#discussion_r177138251
  
    --- Diff: 
core/src/main/java/org/apache/carbondata/core/constants/CarbonCommonConstants.java
 ---
    @@ -1618,6 +1618,11 @@
        */
       public static final String 
CARBON_INVISIBLE_SEGMENTS_PRESERVE_COUNT_DEFAULT = "200";
     
    +  /**
    +   * System older location to store system level data like datamap schema 
and status files.
    +   */
    +  public static final String CARBON_SYSTEM_FOLDER_LOCATION = 
"carbon.system.folder.location";
    --- End diff --
    
    It is better to add default value in this class also


---

Reply via email to