sundapeng opened a new pull request, #7668: URL: https://github.com/apache/paimon/pull/7668
This PR refactors `HadoopCompliantFileIO.delete()` to delegate to a new `protected doDelete()` method. **Motivation:** Subclasses (e.g. JindoFileIO) sometimes need to intercept delete operations (for trash/recycle-bin, audit logging, etc.) while still being able to call the real filesystem delete when needed. Currently there is no clean way to do this — overriding `delete()` loses access to the original implementation. **Changes:** - Extract the filesystem delete logic into `protected doDelete(Path, boolean)` - `delete()` now delegates to `doDelete()` (no behavior change for existing code) - Subclasses can override `delete()` with custom logic and call `doDelete()` for the real delete - Added unit tests verifying the template method pattern works correctly -- 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]
