alamb commented on PR #10288: URL: https://github.com/apache/arrow-rs/pull/10288#issuecomment-5011031668
> AI-assisted tools make it > easy to explore and implement related ideas quickly, but they do not reduce the > reviewer's cognitive load. I agree with this. As you have seen the bandwidth we have to review code in this repo is quite limited (the same I think applies to pretty much every open source project). Thus, while it is unfortunate, expending extra effort as a contributor to make the changes easy to review is often the best way to get a contribution reviewed sooner > I have now narrowed this PR to one behavioral goal: making Mask execution > correct across sparse pages without falling back to selectors. Thank you > Future > optimizations can be evaluated and reviewed separately in focused follow-up > PRs. This is also how I plan to scope my contributions going forward: establish > the behavioral change first, then optimize incrementally with separate evidence. I agree this will hopefully improve the velocity at which we can review and merge your PRs -- 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]
