Github user tillrohrmann commented on the pull request:
https://github.com/apache/flink/pull/1500#issuecomment-172526377
Thanks for your work @chiwanpark. On the whole your changes look good. I
had some minor comments.
I also think that this won't work if the YARN cluster was started with HA
and the first master has died. The reason is that the `.yarn-properties` file
is read to connect to the `JobManager`. But this file is never updated in case
of a new leader. However, this is also a problem which will occur with a normal
Flink job right now.
---
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 [email protected] or file a JIRA ticket
with INFRA.
---
