abhishekrb19 commented on code in PR #16719:
URL: https://github.com/apache/druid/pull/16719#discussion_r1672995759
##########
server/src/main/java/org/apache/druid/server/coordinator/duty/KillUnusedSegments.java:
##########
@@ -127,7 +148,7 @@ public DruidCoordinatorRuntimeParams run(final
DruidCoordinatorRuntimeParams par
if (canDutyRun()) {
return runInternal(params);
} else {
- log.debug(
+ log.info(
Review Comment:
Hm, this is an exit condition I think is worth keeping. Also, there's a
similar log in the `MarkOvershadowedSegmentsAsUnused` which by default runs
every minute (more frequent than the default 30m `KillUnusedSegments` duty
cycle):
https://github.com/apache/druid/blob/master/server/src/main/java/org/apache/druid/server/coordinator/duty/MarkOvershadowedSegmentsAsUnused.java#L74
I will remove it if we feel strongly about it.
--
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]
