Github user manishgupta88 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2565#discussion_r205676555
--- Diff:
datamap/bloom/src/main/java/org/apache/carbondata/datamap/bloom/BloomCoarseGrainDataMap.java
---
@@ -165,9 +180,11 @@ public void initIndexColumnConverters(CarbonTable
carbonTable, List<CarbonColumn
for (CarbonBloomFilter bloomFilter : bloomIndexList) {
boolean scanRequired = bloomFilter.membershipTest(new
Key(bloomQueryModel.filterValue));
if (scanRequired) {
+ String blockletNo =
+ isBlockletCacheLevel ?
String.valueOf(bloomFilter.getBlockletNo()) : "-1";
--- End diff --
I think Bloom dataMap should return the actual blocklet Id. The old
behavior should not be modified based on cache level
---
