mbutrovich commented on issue #1797: URL: https://github.com/apache/iceberg-rust/issues/1797#issuecomment-3458672369
> I agree with [@Sl1mb0](https://github.com/Sl1mb0) in that many abstractions feel like they are taken from the Java implementation and don't feel like idiomatic rust. The crate uses many classes where I think lightweight functions would suffice. This was one of the reasons why I stopped contributing to the repo some time ago. I think this is true of a lot of Rust projects particularly the early days of a “<blah> but in Rust”. Some of that is existing reference implementations, some of that is there just being fewer Rust devs and they might be learning on these projects. I’m not dismissing your concerns at all, just affirming that I’d definitely heard DataFusion described as “Rust written like C++.” We should strive to listen to the rust community (I am still learning here) and what sort of idiomatic Rust gaps exist in the library, and encourage folks to fix them. -- 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]
