carbondata-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ravipesala <...@git.apache.org>
Subject [GitHub] carbondata pull request #1436: [CARBONDATA-1617] Merging carbonindex files w...
Date Mon, 30 Oct 2017 12:38:09 GMT
Github user ravipesala commented on a diff in the pull request:

    https://github.com/apache/carbondata/pull/1436#discussion_r147690043
  
    --- Diff: core/src/main/java/org/apache/carbondata/core/indexstore/BlockletDataMapIndexStore.java
---
    @@ -66,13 +70,17 @@ public BlockletDataMapIndexStore(String carbonStorePath, CarbonLRUCache
lruCache
         segmentLockMap = new ConcurrentHashMap<String, Object>();
       }
     
    -  @Override public BlockletDataMap get(TableBlockIndexUniqueIdentifier tableSegmentUniqueIdentifier)
    +  @Override public BlockletDataMap get(TableBlockIndexUniqueIdentifier identifier)
           throws IOException {
    -    String lruCacheKey = tableSegmentUniqueIdentifier.getUniqueTableSegmentIdentifier();
    +    String lruCacheKey = identifier.getUniqueTableSegmentIdentifier();
         BlockletDataMap dataMap = (BlockletDataMap) lruCache.get(lruCacheKey);
         if (dataMap == null) {
           try {
    -        dataMap = loadAndGetDataMap(tableSegmentUniqueIdentifier);
    +        SegmentIndexFileStore indexFileStore = new SegmentIndexFileStore();
    +        indexFileStore.readAllIIndexOfSegment(CarbonTablePath.getSegmentPath(
    --- End diff --
    
    ok


---

Mime
View raw message