[GitHub] flink issue #4828: [FLINK-4816] [checkpoints] Executions failed from "DEPLOY...

2018-01-09 Thread tony810430
Github user tony810430 commented on the issue: https://github.com/apache/flink/pull/4828 Hi @StephanEwen Let me conclude your comment and clarify some questions in my mind. 1. The original design treated all failures in DEPLOY as restore failure. That is not fair because

[GitHub] flink issue #4828: [FLINK-4816] [checkpoints] Executions failed from "DEPLOY...

2018-01-09 Thread StephanEwen
Github user StephanEwen commented on the issue: https://github.com/apache/flink/pull/4828 I think this approach is not yet sufficient. There can be various reasons why a failure in DEPLOY happens, failed checkpoint restore is only one of the reasons. This also adds some

[GitHub] flink issue #4828: [FLINK-4816] [checkpoints] Executions failed from "DEPLOY...

2017-12-10 Thread tony810430
Github user tony810430 commented on the issue: https://github.com/apache/flink/pull/4828 Hi @StephanEwen Do you have some time to review this PR? Or who can I ask for help if you are too busy? Thank you. ---

[GitHub] flink issue #4828: [FLINK-4816] [checkpoints] Executions failed from "DEPLOY...

2017-10-16 Thread tony810430
Github user tony810430 commented on the issue: https://github.com/apache/flink/pull/4828 Hi @StephanEwen @ramkrish86 I would appreciate it if you could review this PR in your free time. Thank you. ---