GitHub user mxm opened a pull request:

    https://github.com/apache/flink/pull/2514

    [FLINK-4555] add test case to test Yarn cluster shutdown

    In the first commit, it ensures that the cluster shutdown is working as 
expected.
    
    In a second commit, it gives the ResourceManager more time to unregister 
the application. This is not important as Yarn also monitors processes but 
should ensure that everything shuts down cleanly.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/mxm/flink FLINK-4555

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/flink/pull/2514.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #2514
    
----
commit 9f22fe8c5967e4ac77feed0f83e36f5daa7ba74c
Author: Maximilian Michels <m...@apache.org>
Date:   2016-09-19T16:45:05Z

    [FLINK-4555] add test case to test Yarn cluster shutdown

commit 7989dfa73f23fdb262b5d6f466c05579132d9d27
Author: Maximilian Michels <m...@apache.org>
Date:   2016-09-19T16:45:37Z

    wait for ResourceManager to cleanly unregister application
    
    This ensures that the ResourceManager has enough time to unregister the
    application for shutting down.

----


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to