SaketaChalamchala commented on code in PR #9784: URL: https://github.com/apache/ozone/pull/9784#discussion_r2855029902
########## hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/OMDBCheckpointServletInodeBasedXfer.java: ########## Review Comment: @sadanand48 I'm taking another pass at the review and I think we should not handle `NoSuchFileException` for snapshots and OM DB. If an expected file is missing from snapshots or OM checkpoint an error should be thrown and the bootstrap retried, right? We should just handle the exception here for SST backup directory because we know that non-L0 SST files can be pruned from the directory. Let me know what you think. -- 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]
