smengcl opened a new pull request, #11577: URL: https://github.com/apache/ozone/pull/11577
Generated-by: Claude Code (Opus 5.5) ## What changes were proposed in this pull request? `OmSnapshotLocalDataManager#checkOrphanSnapshotVersions` loops over every snapshot queued for the orphan version check and fully loads each snapshot's local data YAML. An `IOException` for one snapshot (for example a YAML that became unreadable after OM startup) ends the whole run, and since that snapshot stays queued, every later run stops at the same place. Orphan versions of the remaining snapshots are never cleaned up. This patch catches `IOException` per snapshot inside the loop, logs it with the snapshot id, keeps that snapshot's count so it is retried in the next run, and continues with the rest. ## What is the link to the Apache JIRA https://issues.apache.org/jira/browse/HDDS-17080 ## How was this patch tested? Added `TestOmSnapshotLocalDataManager#testCheckOrphanSnapshotVersionsContinuesAfterLoadFailure`: two snapshots are queued, one YAML is corrupted, and the test asserts the other snapshot is processed while the failed one stays queued. 🤖 Generated with [Claude Code](https://claude.com/claude-code) -- 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]
