stevenzwu commented on PR #6614: URL: https://github.com/apache/iceberg/pull/6614#issuecomment-1400630461
> Table API assumes that you can have a reference for the table "forever" and you can refresh the table data whenever you want. There should be no restrictions. > This means that the table needs access to an open connection pool until the table is closed. I think that closing the JDBC pool before closing the table is the mistake here. `SerializableTable` is an read-only copy of `Table`. we can't refresh a `SerializableTable`. > As a general rule we expect the Catalog to be an easy static wrapper around the resources needed to access the table snapshot pointer. static wrapper can make the object lifecycle management difficult. E.g., Flink needs to [unload dynamically loaded classes in user code](https://nightlies.apache.org/flink/flink-docs-master/docs/ops/debugging/debugging_classloading/#unloading-of-dynamically-loaded-classes-in-user-code) that often includes connectors. -- 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
