[PR] fix(compute): propagate nested expression serialization errors [arrow-go]

["fallintoplace (via GitHub)" <gi...@apache.org>]

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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] fix(compute): propagate nested expression serialization errors [arrow-go]

["zeroshade (via GitHub)" <gi...@apache.org>]

zeroshade merged PR #1160:
URL: https://github.com/apache/arrow-go/pull/1160


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org