[jira] [Closed] (FLINK-13929) Revisit REST & JM URL

["Till Rohrmann (Jira)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/FLINK-13929?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Till Rohrmann closed FLINK-13929.
---------------------------------
    Resolution: Abandoned

Nobody else seems to have complained about the configuration recently. Hence, I would close this as a non-issue. Feel free to open it again if you object.

> Revisit REST & JM URL 
> ----------------------
>
>                 Key: FLINK-13929
>                 URL: https://issues.apache.org/jira/browse/FLINK-13929
>             Project: Flink
>          Issue Type: Improvement
>          Components: Runtime / Configuration, Runtime / Coordination
>            Reporter: Zili Chen
>            Priority: Minor
>
> Currently we have several issues on URL(i.e., ADDRESS and PORT) configurations of REST(WebMonitor) and JM(DispatcherRMComponent).
>  # Client side code should only retrieve REST PORT but for historical reasons we sometimes pass JM PORT. And this doesn't become a problem because some of them are unused while others JM PORT is incorrectly set with REST PORT value so we do incorrectly twice but conclude in success.
>  # Generally speaking, back to the design of [FLIP-6|https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=65147077], there is no concept named {{WebMonitor}}. The responsibility to communicate with client is covered by {{Dispatcher}}. So it seems no argument to separate {{JobManagerOptions.ADDRESS}} and {{RestOptions.ADDRESS}}. Besides, we unfortunately use different PORT because REST server uses a netty connection while JM requires an actor system which has to bind to another port. Theoretically all message can be passed via the same port, either we handle REST requests in Akka scope or handle RPC in netty scope, so that this "two-port" requirement is hopefully not required then.
>  # nit: Deprecated config {{WebOptions.PORT}} still in use at {{YarnEntrypointUtils.loadConfiguration}}. This should be easily resolved by replaced with {{RestOptions.PORT}}.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)