Github user ravipesala commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1359#discussion_r139059342 --- Diff: core/src/main/java/org/apache/carbondata/core/datamap/dev/DataMap.java --- @@ -31,7 +31,8 @@ /** * It is called to load the data map to memory or to initialize it. */ - void init(String filePath) throws MemoryException, IOException; + void init(String blockletIndexPath, String customIndexPath, String segmentId) --- End diff -- The `filepath` supposed to be either index folder name or index file name, so I don't think this extra information is required here. And also `blockletIndexPath` is not supposed passed as we have carbonIndex exists in other datamap and we supposed to use it.
---