gianm opened a new pull request, #19168:
URL: https://github.com/apache/druid/pull/19168
Main changes:
1) Fix scheduleAtFixedRate to schedule the next run after the prior
run completes, rather than before it starts. The previous logic could
lead to the callable running concurrently with itself when the
scheduled executor is multi-threaded.
2) Align scheduleAtFixedRate and scheduleWithFixedDelay so both of them
re-schedule when the callable throws an exception. Previously
scheduleAtFixedRate did re-schedule but scheduleWithFixedDelay did not.
3) Add DelayMetricEmittingScheduledExecutorService which emits a metric
for scheduling delay. This helps operators understand if a particular
scheduled executor is too overloaded to run tasks on time.
4) Add scheduling delay metrics: overlord/duty/wait/millis,
coordinator/duty/wait/millis, and ingest/reporting/wait/millis.
--
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]
