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

   ### Rationale for this change\n\nStructBuilder allows its field builders to 
diverge from the struct length. A shorter field currently causes a later 
slicing failure, while a longer field silently loses values when the array is 
finalized.\n\n### What changes are included in this PR?\n\n* Require every 
field builder length to match the struct builder length before finalization.\n* 
Report the mismatched field index and both lengths in the panic.\n* Replace the 
existing opaque missing-field failure with the explicit invariant check.\n\n### 
Are these changes tested?\n\nYes. The tests cover both missing and extra field 
values, including the previous silent-discard case. The full arrow/array 
package, assertion build, compute packages, IPC package, and CSV package also 
pass.


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