nandakumar131 opened a new pull request, #7430:
URL: https://github.com/apache/ozone/pull/7430

   ## What changes were proposed in this pull request?
   SCM follower should not log NotLeaderException during Pipeline Report 
processing.
   
   While processing Pipeline Report, SCM follower might log 
`NotLeaderException` when the follower tries to move the Pipeline from 
ALLOCATED state to OPEN state.
   
   This exception can be ignored as this is expected to happen in follower SCM.
   
   We can optionally add `scmContext.isLeader()` check before calling 
`pipelineManager.openPipeline(pipelineID)` this will avoid the exception in the 
first place.
   But, we can still end up with `NotLeaderException` as leader can change 
after `scmContext.isLeader()` check is done.
   
   `scmContext.isLeader()` is an optimization which is not required here.
   
   ## What is the link to the Apache JIRA
   HDDS-11695.
   
   ## How was this patch tested?
   This is logging change and doesn't affect any functionality.
   The code refactoring is tested with existing unit test.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


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

Reply via email to