mdub commented on issue #1816: URL: https://github.com/apache/iceberg-rust/issues/1816#issuecomment-3579183677
I just (this week) started using `iceberg-rust` in a project. So, these are initial impressions. Firstly, many thanks for making and maintaining this! The foundations are solid, and I've been able to get something going relatively quickly. It's also been gratifying to see the activity in both Issues and PRs - this project obviously has an active and interested user-base. To be honest, I'm pretty new to Rust as well, but I _can_ see how patterns have been carried across from the world of Java (and Spark), in a way that seems slightly awkward. One example I've come up against is _configuration_. I'm building a CLI, and focusing on AWS-centric use-cases at the moment, i.e. GlueCatalog, and S3 storage. It feels _quite_ awkward having to construct GlueCatalog and FileIO by providing `props: HashMap<String, String>`. For my use-cases, I would prefer being able to provide an `aws_config:SdkConfig` (or equivalent) to provide common config for Glue and S3 interactions (region, credentials). -- 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]
