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

   ### What changed
   
   NewDenseUnionFromArraysWithFieldCodes now rejects mismatched type ID and 
offset lengths before constructing dense union data.
   
   ### Why
   
   Dense unions use typeIDs.Len() as the logical length while borrowing the 
offsets buffer. If the offsets array is shorter, later access can panic or read 
invalid offset data; if it is longer, trailing offsets are silently ignored.
   
   ### Validation
   
   - go test ./arrow/array -run 'TestUnions/TestMakeDenseUnions' -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]

Reply via email to