Github user uce commented on the pull request:
https://github.com/apache/flink/pull/248#issuecomment-121183870
Yeah, I was also thinking about this.
This will result in a total of 4 combinations:
HA mode | JM address configured | JM address behaviour
------------- | ------------- | -------------
0 | 0 | Use hostname of where the JM is started (new behaviour, this PR)
0 | 1 | Use configured hostname (current behaviour)
1 | 0 | Ignored (current behaviour)
1 | 1 | Ignored (current behaviour)
We would comment out the JM RPC address in the default config and most
cluster setups w/o HA would require one less configuration value.
What do you say? I'm slightly leaning towards adding this, but if your gut
feeling is that this complicates things I'm also happy to close this PR. :)
---
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.
---
