ravipesala commented on a change in pull request #3177:
[CARBONDATA-3337][CARBONDATA-3306] Distributed index server
URL: https://github.com/apache/carbondata/pull/3177#discussion_r279408680
##########
File path:
core/src/main/java/org/apache/carbondata/core/datamap/DataMapStoreManager.java
##########
@@ -483,29 +500,38 @@ public void clearDataMaps(AbsoluteTableIdentifier
identifier) {
*/
public void clearDataMaps(AbsoluteTableIdentifier identifier, boolean
launchJob) {
String tableUniqueName =
identifier.getCarbonTableIdentifier().getTableUniqueName();
- List<TableDataMap> tableIndices = allDataMaps.get(tableUniqueName);
- if (tableIndices == null) {
- String keyUsingTablePath =
getKeyUsingTablePath(identifier.getTablePath());
- if (keyUsingTablePath != null) {
- tableUniqueName = keyUsingTablePath;
- tableIndices = allDataMaps.get(tableUniqueName);
- }
- }
- if (launchJob && tableIndices != null) {
- CarbonTable carbonTable = getCarbonTable(identifier);
+ CarbonTable carbonTable = getCarbonTable(identifier);
Review comment:
This carbon table gets from cache so during clear datamap job we should
clear the table cache as well in executor
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
