paleolimbot commented on code in PR #13397:
URL: https://github.com/apache/arrow/pull/13397#discussion_r915174839
##########
r/R/query-engine.R:
##########
@@ -190,7 +190,7 @@ ExecPlan <- R6Class("ExecPlan",
}
node
},
- Run = function(node) {
+ Run = function(node, as_table = FALSE) {
Review Comment:
I'll see if I can improve the failure mode for `to_duckdb()`, which is as
far as I know the only feature that wouldn't work with a UDF. An warning
message along the lines of "user-defined functions are not supported with
`to_duckdb()`" and collecting to Table first would probably be ok?
Other options include (1) detecting whether or not a plan contains a UDF in
`as_record_batch_reader.arrow_dplyr_query()` or (2) try to figure out an event
loop situation that will work when the plan is executing in the background.
Both of those will take quite some time.
--
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]
