errose28 commented on PR #3465:
URL: https://github.com/apache/ozone/pull/3465#issuecomment-1148216496

   Hi @adoroszlai I have updated the PR. The issue with containers stuck in the 
CLOSING state should be fixed, see the explanation added to the docker-config 
file comments. I have also changed the PR so that the finalization checkpoint 
is written to the `SCMContext` by the `FinalizationStateManager`, where it can 
be read by other managers. This removes a circular dependency, fixes a deadlock 
bug, and makes the code easier to work with IMO.
   
   I will be doing repeated runs of the upgrade acceptance test overnight here 
to test stability, but PTAL in the meantime.
   
   cc @nandakumar131 


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