paleolimbot commented on code in PR #19281:
URL: https://github.com/apache/datafusion/pull/19281#discussion_r2636307539
##########
datafusion/ffi/tests/ffi_udaf.rs:
##########
@@ -126,4 +128,69 @@ mod tests {
Ok(())
}
+
+ /// This test FFI UDFs can be used as inputs to FFI Aggregate UDFs.
+ /// Really this is a test of the Protobuf serialization and deserialization
+ /// using the TaskContextProvider. It can be demonstrated through the
Review Comment:
Is this still using Protobuf or is it doing the thing that used to use
Protobuf but now uses the Foreign context?
--
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]
