Github user asfgit closed the pull request at:
https://github.com/apache/flink/pull/1561
---
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 have this feature
enabled and wishes so, or if the feature is enab
Github user mxm commented on the pull request:
https://github.com/apache/flink/pull/1561#issuecomment-176838676
Thanks for the review @tillrohrmann. Going to merge this soon as it is a
release critical fix.
---
If your project is set up for it, you can reply to this email and have yo
Github user tillrohrmann commented on the pull request:
https://github.com/apache/flink/pull/1561#issuecomment-176737933
As far as I can tell the changes look good to me. Good work @mxm :-)
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user mxm commented on the pull request:
https://github.com/apache/flink/pull/1561#issuecomment-176723782
Test failures unrelated to the changes. Yarn tests pass in this build:
https://travis-ci.org/apache/flink/jobs/105645193
---
If your project is set up for it, you can reply
GitHub user mxm opened a pull request:
https://github.com/apache/flink/pull/1561
[FLINK-3300] fix concurrency bug in YarnJobManager
Adds message passing between Hadoop's async resource manager client and
the YarnJobManager actor.
It helps to view the diff with ignored wh
