alamb commented on PR #10060: URL: https://github.com/apache/arrow-rs/pull/10060#issuecomment-4616409724
Thanks @sjhddh -- I think this looks reasonable but some of the comments seem like they are comments that are related to this PR not that should live in the code. Could you please review the comments and make sure 1. They all make sense to you and refer to parts of the code (rather than changes made in this PR) Can you also explain why you added a new test binary rather than adding a test alongside some of the existing tests? -- 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]
