Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2318#discussion_r189423096
--- Diff:
core/src/main/java/org/apache/carbondata/core/datamap/DataMapStoreManager.java
---
@@ -267,6 +267,8 @@ public TableDataMap getDataMap(CarbonTable table,
DataMapSchema dataMapSchema) {
}
}
}
+ } else {
+ dataMap.clear();
--- End diff --
I think it loses the purpose of cache if you clear for every retrieval.
clear should be called only flush the cache. why do you need to flush cache for
every call?
---
