azagrebin commented on pull request #8952:
URL: https://github.com/apache/flink/pull/8952#issuecomment-701279443


   It makes sense to rebase this PR on new `ActiveResourceManager` refactoring 
PRs for Yarn/Mesos/k8s: #13311 and #13313 because they will be probably merged 
soon in 1.12. That effort changes how `ActiveResourceManager` requests workers, 
now using `ResourceManagerDriver`.
   
   Also, I am not sure that the `FailureRater` is relevant for standalone 
deployment at all. I would consider introducing it only for 
`ActiveResourceManagers`.


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to