devmadhuu commented on PR #4459: URL: https://github.com/apache/ozone/pull/4459#issuecomment-1486363468
> The code looks good to me. But it involves a few moving parts and a test is really needed to ensure different components respond properly, for example, does recon respond to OM properly when the DB update is not successful. @jojochuang - Thanks for the review. Additional UT and assertions have been updated. Pls re-review. -- 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]
