phillipleblanc opened a new pull request, #1297: URL: https://github.com/apache/iceberg-rust/pull/1297
## Which issue does this PR close? - Closes #1296 ## What changes are included in this PR? Makes the IcebergTableProvider::try_new method public that takes an Arc<dyn Catalog> and a TableIdent. It uses that to get the current table metadata when the DataFusion TableProvider is created - but it also stores a reference to the Arc<dyn Catalog>. When the DataFusion TableProvider is asked to scan the table, it uses the catalog to fetch the latest table metadata. This allows the TableProvider to get the latest changes to the Iceberg table, as opposed to being stuck on the snapshot when the table was created. This aligns closer to the expectation of using DataFusion TableProviders, where the scan is expected to scan the latest data. ## Are these changes tested? Covered by the existing integration tests at `crates/integrations/datafusion/tests/integration_datafusion_test.rs` - but I will also add a dedicated test for this. -- 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
