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

Ivan Andika updated HDDS-14336:
-------------------------------
    Description: 
HadoopRpcOMFailoverProxyProvider contains two maps regarding OM proxy info 1) 
omProxyInfos (OMProxyInfo) 2) omProxies (ProxyInfo) from 
OMFailoverProxyProviderBase. This causes some confusion since OMProxyInfo has 
the same name as Hadoop's ProxyInfo, but does not contain any internal proxy 
unlike ProxyInfo. It also makes some use cases require to check two different 
maps, which is tedious.

We can try to unify it by making OMProxyInfo extends ProxyInfo instead and 
remove omProxyInfos from HadoopRpcOMFailoverProxyProvider.

  was:
HadoopRpcOMFailoverProxyProvider contains two maps regarding OM proxy info 1) 
omProxyInfos (OMProxyInfo) 2) omProxies (ProxyInfo) from 
OMFailoverProxyProviderBase. This causes some confusion since OMProxyInfo has 
the same name as Hadoop's ProxyInfo, but does not contain any internal proxy 
unlike ProxyInfo.

We can try to unify it by making OMProxyInfo extends ProxyInfo instead and 
remove omProxyInfos from HadoopRpcOMFailoverProxyProvider.


> Reduce duplication information about OM Proxy in 
> HadoopRpcOMFailoverProxyProvider
> ---------------------------------------------------------------------------------
>
>                 Key: HDDS-14336
>                 URL: https://issues.apache.org/jira/browse/HDDS-14336
>             Project: Apache Ozone
>          Issue Type: Task
>            Reporter: Ivan Andika
>            Assignee: Ivan Andika
>            Priority: Minor
>
> HadoopRpcOMFailoverProxyProvider contains two maps regarding OM proxy info 1) 
> omProxyInfos (OMProxyInfo) 2) omProxies (ProxyInfo) from 
> OMFailoverProxyProviderBase. This causes some confusion since OMProxyInfo has 
> the same name as Hadoop's ProxyInfo, but does not contain any internal proxy 
> unlike ProxyInfo. It also makes some use cases require to check two different 
> maps, which is tedious.
> We can try to unify it by making OMProxyInfo extends ProxyInfo instead and 
> remove omProxyInfos from HadoopRpcOMFailoverProxyProvider.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to