taepper opened a new pull request, #50602: URL: https://github.com/apache/arrow/pull/50602
### Rationale for this change Follow-up to #50247 `GenericNullLikePartition::TranslateTo` translates a partition result from a source range to a target range. This includes pointer arithmetic and does not allow bounds checking for the source and target range, as only the range starts are given. Ideally, we want to pass the entire source and target range to the function, which will allow us to both validate sizes, guarantee that the correct source range is passed, and the resulting `GenericNullLikePartition` will point into the target range. ### What changes are included in this PR? Additional refactor for `GenericNullLikePartition::TranslateTo`, that now allows more `DCHECK`s. Each caller already had a `std::span` object for both source and target, which is now passed in directly. ### Are these changes tested? Re-ran existing test-suite. ### Are there any user-facing changes? No. -- 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]
