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

    https://github.com/apache/carbondata/pull/2778#discussion_r221417633
  
    --- Diff: 
datamap/bloom/src/main/java/org/apache/carbondata/datamap/bloom/BloomCoarseGrainDataMapFactory.java
 ---
    @@ -235,13 +235,13 @@ public DataMapBuilder createBuilder(Segment segment, 
String shardName,
           } else if 
(carbonFile.getName().equals(BloomIndexFileStore.MERGE_INPROGRESS_FILE)) {
             mergeShardInprogress = true;
           } else if (carbonFile.isDirectory()) {
    -        shardPaths.add(carbonFile.getAbsolutePath());
    +        
shardPaths.add(FileFactory.getPath(carbonFile.getAbsolutePath()).toString());
    --- End diff --
    
    FYI:In windows env, the `carbonFile.getAbsolutePath` returns a path with 
path separator '\' while when we loading the cache, the path separator is '/'.


---

Reply via email to