roeap opened a new pull request, #815: URL: https://github.com/apache/iceberg-rust/pull/815
Hey iceberg-rs!, looking to start to more actively contribute to the iceberg ecosystem and trying to grock the codebase, I thought maybe some small contributions might be welcome :). This PR just reduces the surface area for in internal module a little bit, by leveraging partially already defined `TryInto` traits rather then exporting dedicated functions for type conversion. it is IMO a little bit more idiomatic, buf fully understand if you prefer the current way of doing things. -- 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
