Github user asfgit closed the pull request at:
https://github.com/apache/flink/pull/1582
---
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 uce commented on the pull request:
https://github.com/apache/flink/pull/1582#issuecomment-179365728
Thanks Roberto!
---
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
en
Github user uce commented on the pull request:
https://github.com/apache/flink/pull/1582#issuecomment-179365545
Cherry picked to my local `release-0.10` branch and `mvn clean verify`
succeeds.
---
If your project is set up for it, you can reply to this email and have your
reply appea
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/1582#issuecomment-179365564
I think the change is good to merge when travis has passed.
I tried it on GCE:
```
17:36:55,084 INFO org.apache.flink.yarn.YarnTaskManager
GitHub user uce opened a pull request:
https://github.com/apache/flink/pull/1582
[hotfix, yarn] Exit JVM after YARN actor system shut down
Addressed comments from #1576 and waiting for Travis. @rmetzger can you
easily re-test your GCE instance with this PR?
You can merge this pull