niyue opened a new pull request, #38873:
URL: https://github.com/apache/arrow/pull/38873

   ### Rationale for this change
   This PR tries to make Gandiva `FunctionHolder` classes to return 
`arrow::Result` instead of using output parameters, and this tries to address 
the follow up task mentioned in 
https://github.com/apache/arrow/pull/38632#discussion_r1388576545 and makes the 
code slightly simpler
   
   ### What changes are included in this PR?
   A refactoring task to return `arrow::Result` in Gandiva FunctionHolder 
classes
   
   ### Are these changes tested?
   Yes, covered by existing unit tests
   
   ### Are there any user-facing changes?
   No


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