CTTY commented on PR #1972: URL: https://github.com/apache/iceberg-rust/pull/1972#issuecomment-3740903665
> I think we also should create a tracking issue for create external table Yes, I think we currently have `IcebergTableProviderFactory` that can be used for `CREATE EXTERNAL TABLE`, but the problem is that it only creates a read-only static table, and we will need some bigger changes for that. My current thought: we will need to introduce some new catalog props and pass them to `IcebergTableProviderFactory` and then use `CatalogBuilder` to load a `Catalog`, so we switch from using `IcebergStaticTableProvider` to `IcebergTableProvider` in `IcebergTableProviderFactory`. I haven't thought quite thoroughly yet, and will create a tracking issue later, we can move the discussion there -- 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]
