friendlymatthew commented on PR #7878: URL: https://github.com/apache/arrow-rs/pull/7878#issuecomment-3058441512
> > avoid materializing offsets + remove redundant checks > > Honestly, this one feels like a good fast-follow, rather than something that would block the merge? Unless it's super easy/fast to clean up. Perfection as enemy of good and all that. This PR is already a drastic improvement over the current validation. And as @viirya pointed out, this PR adds some validations that were just plain missing before. > > Also, as a separate PR we could hopefully even see benchmark results quantifying the impact of the change. Sounds good. I'll get the benchmarks cleaned up and kick the rest over for another PR -- 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]
