GitHub user liurenjie1024 added a comment to the discussion: Should `IcebergCatalogProvider::try_new` not short-circuit on the first failed `IcebergSchemaProvider::try_new`?
I have concerns about this change, which makes the error hidden from user. I think a more reasonable approach is to load schema/table lazily, e.g. only load them when necessary, and reports the error when loading a corrupted schema/table. But due to the limitation of datafusion's trait definition, we can't do it for now. GitHub link: https://github.com/apache/iceberg-rust/discussions/1651#discussioncomment-14338246 ---- This is an automatically sent email for issues@iceberg.apache.org. To unsubscribe, please send an email to: issues-unsubscr...@iceberg.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
