rdblue commented on PR #6614: URL: https://github.com/apache/iceberg/pull/6614#issuecomment-1399658376
I think that if a catalog is closed, it's reasonable for tables to stop operating as well. The catalog manages its shared resources and if it chooses to share a connection pool with tables then it makes sense for the tables to no longer be able to connect after the pool is closed. Tables should not own their own connection pools, so some resource needs to manage them and the catalog is a good place to do that. I think the problem is that the `TableLoader` is taking ownership of the catalog and closing it. That seems incorrect to me. -- 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: issues-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
