[ https://issues.apache.org/jira/browse/HDDS-11768?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Ivan Andika updated HDDS-11768: ------------------------------- Description: There are a lot of duplicated SCM proxy provider logic in * SCMBlockLocationFailoverProxyProvider * SCMContainerLocationFailoverProxyProvider * SCMSecurityProtocolFailoverProxyProvider * SecretKeyProtocolFailoverProxyProvider It's better to extract the logic to a separate class (e.g parent class) to prevent divergent implementation of this critical logic. SingleSecretKeyProtocolProxyProvider is not included since it is a proxy provider that just communicates with a single SCM. was: There are a lot of duplicated SCM proxy provider logic in * SCMBlockLocationFailoverProxyProvider * SCMContainerLocationFailoverProxyProvider * SCMSecurityProtocolFailoverProxyProvider * SecretKeyProtocolFailoverProxyProvider * SingleSecretKeyProtocolProxyProvider It's better to extract the logic to a separate class (e.g parent class) to prevent divergent implementation of this critical logic. > Extract SCM failover proxy provider logic > ----------------------------------------- > > Key: HDDS-11768 > URL: https://issues.apache.org/jira/browse/HDDS-11768 > Project: Apache Ozone > Issue Type: Sub-task > Components: SCM Client > Reporter: Ivan Andika > Assignee: Ivan Andika > Priority: Major > > There are a lot of duplicated SCM proxy provider logic in > * SCMBlockLocationFailoverProxyProvider > * SCMContainerLocationFailoverProxyProvider > * SCMSecurityProtocolFailoverProxyProvider > * SecretKeyProtocolFailoverProxyProvider > It's better to extract the logic to a separate class (e.g parent class) to > prevent divergent implementation of this critical logic. > SingleSecretKeyProtocolProxyProvider is not included since it is a proxy > provider that just communicates with a single SCM. -- This message was sent by Atlassian Jira (v8.20.10#820010) --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@ozone.apache.org For additional commands, e-mail: issues-h...@ozone.apache.org