Re: [PR] HDDS-13163. Fix DeleteBlockCommandStatus leak in StateContext#cmdStatusMap [ozone]

["chungen0126 (via GitHub)" <gi...@apache.org>]

chungen0126 commented on PR #8542:
URL: https://github.com/apache/ozone/pull/8542#issuecomment-3212007174

   Thanks @xichen01 for the patch. 
   I'd like to ask if the other changes, apart from the modifications within StateContext#getNextCommand, have any specific scenarios where they would occur. If not, I don't think these changes are necessary.
   
   Additionally, the TestStateContext tests are incorrect. You're adding ReplicateContainerCommand to the state machine queue for testing, but only DeleteBlocksCommand is actually added to StateContext#cmdStatusMap.


-- 
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: issues-unsubscribe@ozone.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org