QiuYucheng2003 commented on issue #15031: URL: https://github.com/apache/iceberg/issues/15031#issuecomment-3738211786
Hi@pvary, thank you for the detailed analysis and suggestions! I completely agree with your approach. Replacing the newExitingWorkerPool with standard thread pools and managing their lifecycle explicitly in the close() methods seems to be the correct way to resolve the ClassLoader leak. Unfortunately, I don't have the bandwidth to contribute this fix at the moment. However, since the solution path is now very clear, I hope this discussion helps anyone else who might want to pick this up. -- 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]
