fallintoplace opened a new pull request, #1026: URL: https://github.com/apache/arrow-go/pull/1026
### Rationale for this change DenseUnionBuilder.AppendNulls(0) and AppendEmptyValues(0) still append a child value even though they add no union entries. That hidden mutation shifts the offset of the next real value and leaves inaccessible child data behind. ### What changes are included in this PR? * Make non-positive bulk null and empty appends no-ops. * Preserve the existing shared-child optimization for positive counts. ### Are these changes tested? Yes. The regression test verifies that both zero-count calls leave the union and child builders empty, then confirms the next real value uses offset zero. The full arrow/array package, assertion build, compute packages, and IPC 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]
