Github user jackylk commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2490#discussion_r202706208
--- Diff:
datamap/bloom/src/main/java/org/apache/carbondata/datamap/bloom/AbstractBloomDataMapWriter.java
---
@@ -281,4 +287,13 @@ protected void releaseResouce() {
}
}
+ /**
+ * BloomDataMapBuilder(called when datamap rebuild) is set to true;
+ * BloomDataMapWriter(called when data load) is set to false;
+ *
+ * The reason for this is dict index column is already decoded to
surrogate key when rebuild
+ * but it is still byte array if build datamap when loading
+ */
+ abstract boolean isRebuildProcess();
--- End diff --
@xuchuanyin has merged Writer and Rebuilder for datamap, if you add this
function, then it is still like having different implementations for Writer and
Rebuilder
---
