ASF GitHub Bot commented on FLINK-4555:

GitHub user mxm opened a pull request:


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

    In the first commit, it ensures that the cluster shutdown is working as 
    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:


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.


> Explicitly kill TaskManager on YARN when ApplicationMaster is shutting down
> ---------------------------------------------------------------------------
>                 Key: FLINK-4555
>                 URL: https://issues.apache.org/jira/browse/FLINK-4555
>             Project: Flink
>          Issue Type: Bug
>          Components: YARN Client
>    Affects Versions: 1.1.0
>            Reporter: Robert Metzger
>            Assignee: Maximilian Michels
>            Priority: Critical
> It seems that Flink is not explicitly destroying the TaskManager JVM when the 
> ApplicationMaster is shutting down (when the YARN application is stopping).
> Since this was once in Flink (in 1.0.x) we should add a test case to ensure 
> this feature stays in the code.

This message was sent by Atlassian JIRA

Reply via email to