fallintoplace opened a new pull request, #873: URL: https://github.com/apache/arrow-go/pull/873
### What changed Struct.String now handles child fields that do not have their own validity bitmap when applying the parent struct validity mask. It creates an all-valid bitmap for the child first, clears parent-null positions, and only releases an existing child bitmap when one is present. ### Why When a nullable struct had a required child with no child null bitmap, the masking path could clear bits in a zero-length slice and then release a nil buffer. Calling String on that shape could panic instead of rendering parent-null slots as null. ### Validation - go test ./arrow/array -run TestStructArrayStringerMasksRequiredChildWithoutNullBitmap -count=1 - go test ./arrow/array -count=1 - PARQUET_TEST_DATA=/Users/hoangvu/Code/OSS/arrow-go/parquet-testing/data ARROW_TEST_DATA=/Users/hoangvu/Code/OSS/arrow-go/arrow-testing/data go test ./... -count=1 -- 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]
