cshuo opened a new pull request, #18940: URL: https://github.com/apache/hudi/pull/18940
### Describe the issue this Pull Request addresses `DefaultCleanHandler.close()` should release the underlying Flink write client when the clean handler is closed. The previous implementation incorrectly invoked `writeClient.clean()`, which could trigger an unintended clean operation during handler shutdown instead of closing client resources. ### Summary and Changelog - Updated `DefaultCleanHandler.close()` to call `writeClient.close()` instead of `writeClient.clean()`. - Added `TestDefaultCleanHandler` to verify the close lifecycle delegates to the write client close path without extra client interactions. ### Impact Fixes clean-handler shutdown behavior so closing the handler no longer triggers an unintended clean operation. ### Risk Level low. ### Documentation Update none ### Contributor's checklist - [ ] Read through [contributor's guide](https://hudi.apache.org/contribute/how-to-contribute) - [ ] Enough context is provided in the sections above - [ ] Adequate tests were added if applicable -- 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]
