[ https://issues.apache.org/jira/browse/FLINK-7408?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16121479#comment-16121479 ]
ASF GitHub Bot commented on FLINK-7408: --------------------------------------- Github user zentol commented on a diff in the pull request: https://github.com/apache/flink/pull/4512#discussion_r132426263 --- Diff: flink-yarn/src/main/java/org/apache/flink/yarn/YarnApplicationMasterRunner.java --- @@ -510,8 +511,8 @@ private static Configuration createConfiguration(String baseDirectory, Map<Strin } // if a web monitor shall be started, set the port to random binding - if (configuration.getInteger(JobManagerOptions.WEB_PORT.key(), 0) >= 0) { - configuration.setInteger(JobManagerOptions.WEB_PORT, 0); + if (configuration.getInteger(WebOptions.PORT.key(), 0) >= 0) { --- End diff -- drop call to `key()` > Extract WebRuntimeMonitor options from JobManagerOptions > -------------------------------------------------------- > > Key: FLINK-7408 > URL: https://issues.apache.org/jira/browse/FLINK-7408 > Project: Flink > Issue Type: Improvement > Components: Configuration > Affects Versions: 1.4.0 > Reporter: Till Rohrmann > Assignee: Till Rohrmann > Priority: Trivial > > With the Flip-6 code changes, the {{WebRuntimeMonitor}} won't run exclusively > next to the {{JobManager}}. Therefore, it makes sense to refactor the web > monitor options and moving them from {{JobManagerOptions}} to {{WebOptions}} > and removing the prefix {{jobmanager}}. > This is done as requested by > https://github.com/apache/flink/pull/4492#issuecomment-321271819. -- This message was sent by Atlassian JIRA (v6.4.14#64029)