GlenGeng edited a comment on pull request #1625:
URL: https://github.com/apache/ozone/pull/1625#issuecomment-734081056


   @linyiqun  @bshashikant  Thanks for the review!
   
   Shashi, I found this issue when trying to debug the test case 
TestDeleteWithSlowFollower for SCM HA, which is written by you. You can find 
more details in the jira's description.
   
   The key point is, although this case can pass in master,  after `HDDS-4023. 
Delete closed container after all blocks have been deleted.`, there will be log 
from stale node hander, which is by design not correct.
   
   This case relies on RM to make the container to CLOSING state, so that SCM 
can move container state to CLOSED, and not hold the delete blocks command. 
   
   There are two solutions to avoid the stale node issue, one is shorten the 
frequency of RM in the case TestDeleteWithSlowFollower, the other is the above 
changes.
   
   Actually, this fix would work in the version of Oct 20, which is the latest 
merge from master to HDDS-2823, But seems some change after Oct 20 affect the 
case TestDeleteWithSlowFollower again.


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

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