zeroshade opened a new pull request, #1532:
URL: https://github.com/apache/arrow-go/pull/1532

   ### Rationale for this change
   
   `Struct.String` panics with `index out of range [0] with length 0` when a 
struct has a run-end encoded child and the struct's validity bitmap differs 
from the child's, for example after appending a null struct. The array is 
valid; only printing fails.
   
   ### What changes are included in this PR?
   
   `String` combines each child's validity bitmap with the parent's by 
replacing buffer 0 of the child's data. Run-end encoded arrays have no buffers, 
so there is no bitmap to combine. Such children are now printed as they are, 
the same way union children already are. This also covers extension types with 
run-end encoded storage.
   
   ### Are these changes tested?
   
   Yes.
   
   ### Are there any user-facing changes?
   
   Printing a struct array with a run-end encoded child no longer panics.
   
   Closes #1531.


-- 
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]

Reply via email to