fallintoplace opened a new pull request, #1160: URL: https://github.com/apache/arrow-go/pull/1160
### Rationale for this change SerializeExpr ignores errors returned while visiting call arguments. A nested unsupported literal can therefore produce a successful result with incomplete metadata, and scalar columns allocated before the error are not released. ### What changes are included in this PR? Propagate errors from nested arguments and release all temporary scalar columns on both success and failure paths. ### Are these changes tested? Yes. The regression test places a valid scalar before an unsupported nested array literal and checks both the returned error and allocator cleanup. The full arrow/compute package suite passes. ### Are there any user-facing changes? Nested serialization failures are now returned to the caller instead of being silently ignored. -- 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]
