[
https://issues.apache.org/jira/browse/FLINK-31518?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17703618#comment-17703618
]
Prabhu Joseph commented on FLINK-31518:
---------------------------------------
[~huwh] It is a thread launched by AdaptiveScheduler which requires the
webMonitorAddress. Currently there is no way to fetch the webMonitorAddress at
AdaptiveScheduler initialization. There are two options we got
Option 1: Pass the webMonitorAddress all the way from
DefaultDispatcherResourceManagerComponentFactory -> Dispatcher -> JobMaster ->
DefaultSlotPoolServiceSchedulerFactory -> AdaptiveScheduler. There are so many
factories, implementations, interface apis need to be changed to add
webMonitorAddress parameter.
Option 2: Fix ClusterRestEndpointLeaderRetriever to provide the right
webMonitorAddress when rest.port is a range.
The Option 1 does not look clean as so much api change required. I have to
analyze further on Option 2. There is one more option we thought which is to
have a shared context class with all the configs/informations that Dispatcher,
JobMaster, SlotPoolService, AdaptiveScheduler can use. This requires change in
only the context class if any new information has to be added later.
> HighAvailabilityServiceUtils resolves wrong webMonitorAddress in Standalone
> mode
> --------------------------------------------------------------------------------
>
> Key: FLINK-31518
> URL: https://issues.apache.org/jira/browse/FLINK-31518
> Project: Flink
> Issue Type: Bug
> Components: Runtime / Configuration, Runtime / REST
> Affects Versions: 1.16.0, 1.17.0, 1.16.1, 1.17.1
> Reporter: Vineeth Naroju
> Priority: Major
>
> {{HighAvailabilityServiceUtils#createHighAvailabilityServices()}} in
> {{HighAvailabilityMode.NONE}} mode uses
> {{HighAvailabilityServiceUtils#getWebMonitorAddress()}} to get web monitor
> address.
> {{HighAvailabilityServiceUtils#getWebMonitorAddress()}} reads only from
> {{rest.port}} in {{flink-conf.yaml}} . If {{rest.port}} is not enabled, then
> it returns {{0}} port number. It should dynamically fetch port number if
> {{rest.port}} is disabled.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
