osscm commented on PR #15492: URL: https://github.com/apache/iceberg/pull/15492#issuecomment-4127681940
> I think we might be solving different issues here. We will definitely need the solution in #15312 to get rid of the issue described in #15031. yes @mxm, both PR has similar logic, I think I had this PR internally, so shared here..., and there are a few difference in terms of handing TP shutDown logic, having timeout for each TP, synchronization in shutDown, having a separate ThreadPool lifecycle manager, had test cases, flag for managing shutdownHook, we can also merge the code in one PR, specially the shutdown method/logic and keeping a TP lifecycle manager. -- 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]
