fallintoplace opened a new pull request, #946: URL: https://github.com/apache/arrow-go/pull/946
### Rationale for this change `GetBatchBools` rounded the remaining value count down to a whole byte inside a loop that also handled one-to-seven trailing values. That produced a zero-length read without advancing the loop. ### What changes are included in this PR? Run the bulk path only while at least eight values remain, then decode the trailing bits through the existing buffered path. ### Are these changes tested? Yes. The regression test covers lengths 0 through 17 from both aligned and unaligned starting offsets. `go test ./parquet/internal/utils` passes. ### Are there any user-facing changes? Boolean batches shorter than a byte, or with a trailing partial byte, no longer stall. -- 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]
