Github user xuchuanyin commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2211#discussion_r183411036
--- Diff:
core/src/main/java/org/apache/carbondata/core/datamap/DataMapStoreManager.java
---
@@ -92,6 +95,30 @@ private DataMapStoreManager() {
return dataMaps;
}
+ /**
+ * It only gives the visible datamaps
+ */
+ private List<TableDataMap> getAllVisibleDataMap(CarbonTable carbonTable)
throws IOException {
+ CarbonSessionInfo sessionInfo =
ThreadLocalSessionInfo.getCarbonSessionInfo();
+ List<TableDataMap> allDataMaps = getAllDataMap(carbonTable);
+ Iterator<TableDataMap> dataMapIterator = allDataMaps.iterator();
+ while (dataMapIterator.hasNext()) {
+ TableDataMap dataMap = dataMapIterator.next();
+ String dbName = carbonTable.getDatabaseName();
+ String tableName = carbonTable.getTableName();
+ String dmName = dataMap.getDataMapSchema().getDataMapName();
+ boolean isDmVisible = sessionInfo.getSessionParams().getProperty(
+ String.format("%s%s.%s.%s",
CarbonCommonConstants.CARBON_DATAMAP_VISIBLE,
--- End diff --
yes, it is temporary, no need to persist it.
---
