maytasm commented on a change in pull request #11426:
URL: https://github.com/apache/druid/pull/11426#discussion_r666804033
##########
File path:
server/src/main/java/org/apache/druid/server/coordinator/duty/CompactSegments.java
##########
@@ -213,11 +212,11 @@ public DruidCoordinatorRuntimeParams
run(DruidCoordinatorRuntimeParams params)
}
} else {
LOG.info("compactionConfig is empty. Skip.");
-
updateAutoCompactionSnapshotWhenNoCompactTaskScheduled(currentRunAutoCompactionSnapshotBuilders);
+ autoCompactionSnapshotPerDataSource.set(new HashMap<>());
Review comment:
autoCompactionSnapshotPerDataSource are built new from empty map every
time the auto compaction runs. At the end of the auto compaction we do a atomic
swap to replace the old map (which has in-active datasource) with the newly
built map (which does not have in-active datasource)
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
