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

    https://github.com/apache/carbondata/pull/2375#discussion_r196305431
  
    --- Diff: 
core/src/main/java/org/apache/carbondata/core/metadata/schema/table/CarbonTable.java
 ---
    @@ -1032,5 +1073,32 @@ public static void 
updateTableByTableInfo(CarbonTable table, TableInfo tableInfo
         }
         table.hasDataMapSchema =
             null != tableInfo.getDataMapSchemaList() && 
tableInfo.getDataMapSchemaList().size() > 0;
    +    setLocalDictInfo(table, tableInfo);
    +  }
    +
    +  /**
    +   * This method sets whether the local dictionary is enabled or not, and 
the local dictionary
    +   * threshold, if not defined default value are considered.
    +   * @param table
    +   * @param tableInfo
    +   */
    +  private static void setLocalDictInfo(CarbonTable table, TableInfo 
tableInfo) {
    --- End diff --
    
    I guess this method not required


---

Reply via email to