zhangwl9 commented on code in PR #3659:
URL: https://github.com/apache/amoro/pull/3659#discussion_r2221558832
##########
amoro-ams/src/main/java/org/apache/amoro/server/scheduler/inline/DanglingDeleteFilesCleaningExecutor.java:
##########
@@ -41,7 +43,7 @@ protected DanglingDeleteFilesCleaningExecutor(TableService
tableService, int poo
@Override
protected long getNextExecutingTime(DefaultTableRuntime tableRuntime) {
- return INTERVAL;
+ return INTERVAL + getExecutorDelay();
Review Comment:
@klion26 I found that getStartDelay() is only called when the executor is
started for the first time, and for subsequent cyclic execution of the
executor, getNextExecutingTime() is called, intending that the trigger time of
the executor at both the first start and the cyclic execution has a randomness
to it, so I add the random value here as well
--
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]
