TisonKun created FLINK-13929:
--------------------------------
Summary: 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: TisonKun
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.2#803003)
