liurenjie1024 commented on PR #1580: URL: https://github.com/apache/iceberg-rust/pull/1580#issuecomment-3297455675
> Hi, this change (particularly removing `RestCatalogConfig`) made the API even more unergonomic than it was before. What's the rational for moving from a typed interface to an untyped one (hashmap of strings)? At the risk of being glib, this reads more like Java than Rust 🙃 The motivation behind the refactoring is to have an unified way to load catalogs. We had some discussions about type safe configs, but we need to a design for it. The old `RestCatalogConfig` approach is not the desired approach for type safe configs. -- 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
