jlambatl commented on PR #2396: URL: https://github.com/apache/iceberg-rust/pull/2396#issuecomment-4986064492
@manuzhang I've reviewed your feedback and confirmed that the concerns you raised do exist. I've manually refactored the code and addressed the feedback. I've then had Codex and Gemini review it, which found some problems that have now been resolved. This Pr has become much larger than i anticipated, however I hope that it is now in a state that is more maintainable moving forward. Keen to hear feedback, thanks in advance. -- 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]
