fallintoplace opened a new pull request, #1005: URL: https://github.com/apache/arrow-go/pull/1005
### Rationale for this change byteReader updated its cached position even when the underlying seek failed. This could make Peek observe a different position from Read. Discard also returned EOF when it successfully consumed exactly the remaining bytes. ### What changes are included in this PR? Only update the cached position after a successful seek and distinguish exact exhaustion from a short discard. Add focused tests for both cases. ### Are these changes tested? Yes. go test ./internal/utils ### Are there any user-facing changes? Parquet byte-buffer readers now keep Peek and Read synchronized after seek errors and match buffered-reader discard semantics at exact EOF. -- 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]
