westonpace commented on PR #12590:
URL: https://github.com/apache/arrow/pull/12590#issuecomment-1109529315

   > @westonpace I have a few questions regarding this.
   > 
   > Here, eventhough we are not exposing whether the value is a scalar or an 
array, but we must say what is the data type of the input. Or should we relax 
this too?
   
   @vibhatha 
   
   I think we should relax this.  We can completely hide the "input type" 
concept from the user and just take a data type.  So it becomes...
   
   ```
   pc.register_scalar_function(some_function,
                                   func_name,
                                   function_doc,
                                   {
                                       "left": pa.int64(),
                                       "right": pa.int64()
                                   },
                                   pa.int64())
   ```
   
   If we later decide to add the input type concept back in that will be fine.  
`pa.int64()` will mean `ANY` and `pc.InputType.array(pa.int64())` will be 
`ARRAY` only.


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