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 other recovery PRs as this one contains all respective
additions?
I've tested it out locally and it works fine. I think moving this to the
job client actor is better than having it the repeated retrieval in the job
client. And the addition with the submission and connection timeouts are also
very good.
---
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.
---
