Github user uce commented on the pull request:
https://github.com/apache/flink/pull/1249#issuecomment-147070413
+1 good to merge. We have to remove dc9daef before merging though (that was
a temporary work around, because we didn't support blocking HA submissions.
Should I close the
Github user tillrohrmann commented on the pull request:
https://github.com/apache/flink/pull/1249#issuecomment-147086347
Thanks for the review @uce. Thanks for pointing out the temporary commit. I
will remove it when I merge this PR. No don't close the PRs. I'll commit them
GitHub user tillrohrmann opened a pull request:
https://github.com/apache/flink/pull/1249
[FLINK-2804] [runtime] Add blocking job submission support for HA
The JobClientActor is now repsonsible for receiving the JobStatus updates
from
a newly elected leader. It uses the