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

    https://github.com/apache/carbondata/pull/2666#discussion_r213334033
  
    --- Diff: 
core/src/main/java/org/apache/carbondata/core/datamap/DataMapStoreManager.java 
---
    @@ -341,6 +348,18 @@ public TableDataMap getDataMap(CarbonTable table, 
DataMapSchema dataMapSchema) {
         return dataMap;
       }
     
    +  private String getKeyUsingTablePath(String tablePath) {
    +    if (tablePath != null) {
    +      // Try get using table path
    +      for (Map.Entry<String, String> entry : tablePathMap.entrySet()) {
    +        if (new Path(entry.getValue()).equals(new Path(tablePath))) {
    --- End diff --
    
    I think we can directly use tablePathMap.get(tablePath).  new Path does not 
change comparison


---

Reply via email to