Github user ajantha-bhat commented on a diff in the pull request:

    https://github.com/apache/carbondata/pull/2322#discussion_r189509573
  
    --- Diff: 
core/src/main/java/org/apache/carbondata/core/indexstore/blockletindex/BlockletDataMapFactory.java
 ---
    @@ -191,7 +143,7 @@ private boolean 
isSameColumnSchemaList(List<ColumnSchema> indexFileColumnList,
           return detailedBlocklets;
         }
         Set<TableBlockIndexUniqueIdentifier> identifiers =
    -        getTableBlockIndexUniqueIdentifiers(segment);
    +        getTableBlockIndexUniqueIdentifiers(segment, 
this.getCarbonTable().getTableUniqueName());
    --- End diff --
    
    Can do that. But..
    
    getTableBlockIndexUniqueIdentifiers is depednent on segmentMap of 
BlockletDataMapFactory. DeCoupling segmentMap from BlockletDataMapFactory needs 
more effrot. can handle in separate PR.
    



---

Reply via email to