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.
---
