niyue commented on PR #38632:
URL: https://github.com/apache/arrow/pull/38632#issuecomment-1802099367

   @js8544 @kou 
   As discussed in the recent PR https://github.com/apache/arrow/pull/38116, I 
submitted a follow up PR to further enhance Gandiva's external function 
registration capability, allowing registering `C interface functions` (`stub 
function` called by Gandiva internally) in this PR. Could you please help to 
review? Thanks. 
   
   ### notes
   * The name `C interface function` is different with `stub function` since I 
think `stub function` may not be easily understood externally, and I am looking 
for something equivalent to `LLVM IR based function` but I am not sure if there 
is a better term than `C interface function`, please advice.
   * I am not sure if I need to post a new discussion thread to the mailing 
list since it was discussed previously for PR 38116, and this PR is a smaller 
enhancement to the previous one. Please let me know if I need to post a new 
discussion thread for this. Thanks.


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