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

    https://github.com/apache/carbondata/pull/2202#discussion_r183230534
  
    --- Diff: 
core/src/main/java/org/apache/carbondata/core/util/CarbonUtil.java ---
    @@ -2942,5 +2947,31 @@ public static String 
getBlockId(AbsoluteTableIdentifier identifier, String fileP
         return blockId;
       }
     
    +  /**
    +   * methods returns whether the feature is supported for the 
corresponding datamap or not
    +   * @param carbonTable
    +   * @param feature
    +   * @return
    +   */
    +  public static boolean validateFeatureForDatamap(CarbonTable carbonTable, 
FeaturesList feature) {
    +    boolean isSupported = false;
    +    try {
    +      List<TableDataMap> datamaps = 
DataMapStoreManager.getInstance().getAllDataMap(carbonTable);
    +      if (!datamaps.isEmpty()) {
    +        for (TableDataMap dataMap : datamaps) {
    +          DataMapFactory factoryClass = DataMapStoreManager.getInstance()
    --- End diff --
    
    I suggest to change `DataMapFactory` class name to `IndexDataMap`


---

Reply via email to