[GitHub] [arrow-ballista] thinkharderdev commented on a diff in pull request #667: Add executor terminating status for graceful shutdown

["thinkharderdev (via GitHub)" <gi...@apache.org>]

thinkharderdev commented on code in PR #667:
URL: https://github.com/apache/arrow-ballista/pull/667#discussion_r1103796298


##########
ballista/scheduler/src/state/executor_manager.rs:
##########
@@ -85,6 +85,10 @@ impl ExecutorReservation {
 /// to be dead.
 pub const DEFAULT_EXECUTOR_TIMEOUT_SECONDS: u64 = 180;
 
+// TODO move to configuration file
+/// Interval check for expired or dead executors
+pub const EXPIRE_DEAD_EXECUTOR_INTERVAL_SECS: u64 = 15;

Review Comment:
   This is changed to poll for dead executors more frequently



-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org