CTTY commented on issue #2021: URL: https://github.com/apache/iceberg-rust/issues/2021#issuecomment-3757607686
Hi @liurenjie1024 , thanks for the inputs! I think your example aligns with my PR #2022 > we need to add a check in IcebergSchemaProvider::register_table to refuse non iceberg table. I agree, we will have to allow `MemTable` as well since datafusion has to use `MemTable` for `CREATE TABLE` sql I will add the change to block other table types in the `CREATE EXTERNAL TABLE` PR -- 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]
