nastra commented on PR #14590: URL: https://github.com/apache/iceberg/pull/14590#issuecomment-6602616079
> * Do we really want to tie the pool to the Catalog? Catalogs are often recreated, and we don't handle lifecycle for them. I do share the same concern as you, so I'm hesitant on introducing closable catalogs that close the pool > * Shall we have our own lifecycle for the AuthSession threadpool? That makes sense to me and should fix the underlying issue -- 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]
