milenkovicm opened a new pull request, #1175:
URL: https://github.com/apache/datafusion-ballista/pull/1175
# Which issue does this PR close?
Closes #1174.
# Rationale for this change
Rest api reports correct number of registered executors in case of
`TaskSchedulingPolicy::PullStaged`
# What changes are included in this PR?
In case `TaskSchedulingPolicy::PullStaged` is configured, executor
registration is slightly different and no heartbeat are sent from executor to
client. rest api method returning executors will call method which will first
call heartbeats status and then consult registered executor list. This change
will revert that order.
As heartbeat is not available there is no information about when is executor
last seen, so rest api will return None in that case. This behaviour can be
changed, we can add additional field in ExecutorMetadata indicating when was
the last time executor called scheduler.
# Are there any user-facing changes?
No
--
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-unsubscr...@datafusion.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org
For additional commands, e-mail: github-h...@datafusion.apache.org
