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

Till Rohrmann updated FLINK-13929:
----------------------------------
    Priority: Minor  (was: Major)

> 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)

Reply via email to