xBis7 commented on PR #3741:
URL: https://github.com/apache/ozone/pull/3741#issuecomment-1252777196

   @errose28 Thanks for your comments, I've made the changes and also modified 
the tests to use the new code path. Let me know what you think.
   
   > One other thing to check is what happens with schema V3 when the container 
is moved to the tmp dir but the RocksDB update fails. Will the datanode restart 
cleanly? What happens to the RocksDB entries for that container?
   
   I will be looking at that next.
   
   


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