Github user tillrohrmann commented on the issue:
https://github.com/apache/flink/pull/3942
Really good work @EronWright. I've completed the review and rebased it on
the latest master. While doing the review I applied one fix for a race
condition in the tests, another fix to address
Github user tillrohrmann commented on the issue:
https://github.com/apache/flink/pull/3942
Sorry for the long delay Eron. I've started reviewing the code last week.
I'm not completely done yet, but so far things look good. As a follow up we
could think about how we can remove the
Github user EronWright commented on the issue:
https://github.com/apache/flink/pull/3942
@tillrohrmann thoughts on this? I'm open to anything.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not
Github user tillrohrmann commented on the issue:
https://github.com/apache/flink/pull/3942
Thanks for your contribution @EronWright. I will try to review and merge
this PR this week.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user EronWright commented on the issue:
https://github.com/apache/flink/pull/3942
This is a fairly straightforward port of the RM code to the FLIP-6
architecture.
Follow-ups to occur in future PRs include:
- adding support for releasing the worker
-
