[
https://issues.apache.org/jira/browse/HDDS-972?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16763192#comment-16763192
]
Bharat Viswanadham commented on HDDS-972:
-----------------------------------------
Hi [~hanishakoneru]
Thank You for the patch. Overall LGTM. Few comments.
# In loginUser, we call InetSocketAddress socAddr =
OmUtils.getOmAddress(conf), but still getOMAddress is not modified to address
HA scenario. I think we should update it here.
As still, getOmRpcAddress is just reading as below
*final Optional<String> host = getHostNameFromConfigKeys(conf,*
*OZONE_OM_ADDRESS_KEY);*
2. One question in MiniOzoneHAClusterImpl, we are generating ports randomly
basePort = 10000 + RANDOM.nextInt(1000) * 4; But here we are not checking
whether ports are free or not. Otherwise, when we start MiniOzoneHAClusterImpl
we get an error during start right?
> Add support for configuring multiple OMs
> ----------------------------------------
>
> Key: HDDS-972
> URL: https://issues.apache.org/jira/browse/HDDS-972
> Project: Hadoop Distributed Data Store
> Issue Type: Sub-task
> Reporter: Hanisha Koneru
> Assignee: Hanisha Koneru
> Priority: Major
> Attachments: HDDS-972.000.patch, HDDS-972.001.patch,
> HDDS-972.002.patch, HDDS-972.003.patch, HDDS-972.004.patch
>
>
> For OM HA, we would need to run multiple (atleast 3) OM services so that we
> can form a replicated Ratis ring of OMs. This Jira aims to add support for
> configuring multiple OMs.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
