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 [email protected]. To unsubscribe, please send an email to: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
