[ 
https://issues.apache.org/jira/browse/HDDS-12648?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17936942#comment-17936942
 ] 

Ayush Saxena commented on HDDS-12648:
-------------------------------------

there is a check {{ OzoneManagerRatisUtils.checkLeaderStatus(ozoneManager)}} in 
the method calling, can see if that itself is sufficient & if we can drop this 
check over here rather than fixing

> Fix checkLeaderStatus in removeOMFromRatisRing and addOMToRatisRing
> -------------------------------------------------------------------
>
>                 Key: HDDS-12648
>                 URL: https://issues.apache.org/jira/browse/HDDS-12648
>             Project: Apache Ozone
>          Issue Type: Bug
>            Reporter: Ayush Saxena
>            Priority: Minor
>
> Currently the _checkLeaderStatus_ check does nothing, it return the leader 
> status & ignores
> [https://github.com/apache/ozone/blob/57fda0c32704b76b78152f5b16bc9a7c923167ac/hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/ratis/OzoneManagerRatisServer.java#L360]
>  
> If that is required, maybe should change to
>  
> {code:java}
> ozoneManager.checkLeaderStatus(){code}
>  



--
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