len548 opened a new pull request, #11016: URL: https://github.com/apache/ozone/pull/11016
## What changes were proposed in this pull request? On legacy buckets (non-FSO path), `delete()` and `rename()` in `TrashOzoneFileSystem` could report success even when the underlying OM write failed: 1. `RenameIterator` / `DeleteIterator` swallowed errors — `submitRequest()` failures were caught, logged, and iteration continued as if nothing failed. 2. Return values were ignored — top-level `delete()` / `rename()` always returned true, even when `iterate()` returned false. 3. Null OMRequest was not handled — if `createOMRequest()` failed, a null request could be passed to `submitRequest()` or fail without propagating failure to the caller. This fix propagates these delete/rename failures from the LEGACY trash iterator path instead of always returning true. ## What is the link to the Apache JIRA https://issues.apache.org/jira/browse/HDDS-16163 ## How was this patch tested? Add UT written by Cursor with Composer 2.5 model -- 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]
