Rich-T-kid opened a new pull request, #12212: URL: https://github.com/apache/arrow-rs/pull/12212
# Which issue does this PR close? Closes #12210. # Rationale for this change The sparse ByteView indexed-copy path added in #11174 checks the destination buffer boundary for every selected non-inline value, even though it already computes the total selected byte length before copying. This PR is stacked on #11174 at `f4544e6d23`. Its diff will reduce to the single optimization commit when #11174 merges. # What changes are included in this PR? Selections that fit in one output buffer use the existing byte-total calculation to copy without per-value boundary checks. Selections larger than the buffer limit are partitioned into contiguous groups first, then each group is copied without boundary checks in the copy loop. The existing overflow checks and maximum buffer size are preserved. # Are these changes tested? The split-buffer tests cover exact buffer boundaries, inline values between non-inline values, multiple output buffers, and full ByteView validation. The hosted `coalesce_kernels` benchmark compares this branch with the exact #11174 head. # Are there any significant 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]
