{"action":"created","issue":{"url":"https://api.github.com/repos/apache/flink/issues/2276","repository_url":"https://api.github.com/repos/apache/flink","labels_url":"https://api.github.com/repos/apache/flink/issues/2276/labels{/name}","comments_url":"https://api.github.com/repos/apache/flink/issues/2276/comments","events_url":"https://api.github.com/repos/apache/flink/issues/2276/events","html_url":"https://github.com/apache/flink/pull/2276","id":166802399,"number":2276,"title":"[FLINK-4201] [runtime] Forward suspend to checkpoint coordinator","user":{"login":"uce","id":1756620,"avatar_url":"https://avatars.githubusercontent.com/u/1756620?v=3","gravatar_id":"","url":"https://api.github.com/users/uce","html_url":"https://github.com/uce","followers_url":"https://api.github.com/users/uce/followers","following_url":"https://api.github.com/users/uce/following{/other_user}","gists_url":"https://api.github.com/users/uce/gists{/gist_id}","starred_url":"https://api.github.com/users/uce/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/uce/subscriptions","organizations_url":"https://api.github.com/users/uce/orgs","repos_url":"https://api.github.com/users/uce/repos","events_url":"https://api.github.com/users/uce/events{/privacy}","received_events_url":"https://api.github.com/users/uce/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2016-07-21T12:02:13Z","updated_at":"2016-07-21T13:50:31Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/apache/flink/pulls/2276","html_url":"https://github.com/apache/flink/pull/2276","diff_url":"https://github.com/apache/flink/pull/2276.diff","patch_url":"https://github.com/apache/flink/pull/2276.patch"},"body":"Suspended jobs were leading to shutdown of the checkpoint coordinator and hence removal of checkpoint state. For standalone recovery mode this is OK as no state can be recovered anyways (unchanged in this PR). For HA though this lead to removal of checkpoint state, which we\r\nactually want to keep for recovery.\r\n\r\nWe have the following behaviour now:\r\n\r\n\r\nJobState | Standalone | High Availability\r\n-----------|------------|-------------------\r\n SUSPENDED |  Discard   |       Keep\r\n FINISHED/FAILED/CANCELED |  Discard   |     Discard"},"comment":{"url":"https://api.github.com/repos/apache/flink/issues/comments/234258943","html_url":"https://github.com/apache/flink/pull/2276#issuecomment-234258943","issue_url":"https://api.github.com/repos/apache/flink/issues/2276","id":234258943,"user":{"login":"StephanEwen","id":1727146,"avatar_url":"https://avatars.githubusercontent.com/u/1727146?v=3","gravatar_id":"","url":"https://api.github.com/users/StephanEwen","html_url":"https://github.com/StephanEwen","followers_url":"https://api.github.com/users/StephanEwen/followers","following_url":"https://api.github.com/users/StephanEwen/following{/other_user}","gists_url":"https://api.github.com/users/StephanEwen/gists{/gist_id}","starred_url":"https://api.github.com/users/StephanEwen/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/StephanEwen/subscriptions","organizations_url":"https://api.github.com/users/StephanEwen/orgs","repos_url":"https://api.github.com/users/StephanEwen/repos","events_url":"https://api.github.com/users/StephanEwen/events{/privacy}","received_events_url":"https://api.github.com/users/StephanEwen/received_events","type":"User","site_admin":false},"created_at":"2016-07-21T13:50:31Z","updated_at":"2016-07-21T13:50:31Z","body":"I think that change is good in \"our\" philosophy, just wondering about the following implication:\r\n\r\nSome people want to run HA setups without ZooKeeper, simply using an external service to make sure that the JobManager is restarted. The latest completed checkpoint is found via a well-defined path in the checkpoint storage (rather than a well defined path in ZooKeeper).\r\n\r\nThat works as an HA setup (with the exception of being susceptible to \"split brain\" behavior in the presence of network partitions. \r\n\r\nWould we interfere with such a setup when removing checkpoints on \"suspend\" in \"standalone\" mode?\r\n\r\nBTW: We should really find a new name for \"standalone\" more ;-) The term is overloaded."}}
{"actor":{"display_login":"StephanEwen"}}