zeroshade opened a new pull request, #634:
URL: https://github.com/apache/arrow-go/pull/634
### Rationale for this change
Fixes #625
### What changes are included in this PR?
When all values appended to a BinaryBuilder are empty (zero-length byte
slices), they were incorrectly being treated as NULL values instead of valid
empty slices. This occurred because the underlying buffer builder returned a
buffer created from nil bytes, which caused slicing operations to return nil.
The fix ensures that when no data buffer is allocated (length = 0), we
create a buffer with an empty slice []byte{} instead of nil, allowing proper
distinction between NULL and empty values.
### Are these changes tested?
Yes a test is added.
### Are there any user-facing changes?
Just fixing the behavior.
--
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]
