A-makarim opened a new pull request, #51240:
URL: https://github.com/apache/arrow/pull/51240
### Rationale for this change
Boolean values are bit-packed. When copying values from a sliced boolean
array,
the copy path did not include the array's slice offset. This caused
`fill_null_forward`, `fill_null_backward`, and `replace_with_mask` to read
values from earlier positions in the parent array.
### What changes are included in this PR?
- Apply `ArraySpan::offset` to the bit index when copying boolean values from
an `ArraySpan`.
- Add regression coverage for sliced boolean inputs in `replace_with_mask`,
`fill_null_forward`, and `fill_null_backward`.
### Are these changes tested?
Yes. I ran `arrow-compute-vector-test` locally.
### Are there any user-facing changes?
Only a bugfix.
### This PR contains a "Critical Fix".
This fixes a bug that caused compute operations on sliced boolean arrays with
a non-zero offset to produce incorrect values.
* GitHub Issue: #51223
--
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]
