andygrove opened a new issue, #6177:
URL: https://github.com/apache/datafusion-comet/issues/6177

   **Arity.** `CometNativeUDF.register` installs a Spark catalog stub so that 
SQL and DataFrame name
   resolution succeed, and that stub is built from `functions.udf` with a 
fixed-arity Scala lambda, one
   case per arity from 0 to 4. A UDF with five or more arguments is rejected at 
registration. Neither
   the ABI nor the native adapter has an arity limit; only the stub does. 
Spark's `UDF5` through
   `UDF22` interfaces (or building the `ScalaUDF` directly) would remove the 
cap.
   
   **`registerAll`.** A library can export any number of kernels, but each has 
to be registered by
   name with its own signature. A `registerAll(spark, libraryPath)` that 
registered every exported
   kernel would need the library to report each kernel's signature, which ties 
into #5597 (derive the
   return type from the library rather than asking the caller to declare it).
   
   Follow-up to #4459.
   


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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to