jonkeane commented on a change in pull request #12357:
URL: https://github.com/apache/arrow/pull/12357#discussion_r804190644
##########
File path: r/tests/testthat/_snaps/dplyr-funcs-datetime.md
##########
@@ -0,0 +1,15 @@
+# extract tz
+
+ Code
+ compare_dplyr_binding(.input %>% mutate(timezone_y = tz(x), timezone_z =
tz(z),
+ timezone_w = tz(w), timezone_v = tz(v)) %>% collect(), df)
+ Warning <simpleWarning>
+ tz(): Don't know how to compute timezone for object of class integer;
returning "UTC". This warning will become an error in the next major version of
lubridate.
+ tz(): Don't know how to compute timezone for object of class numeric;
returning "UTC". This warning will become an error in the next major version of
lubridate.
+ tz(): Don't know how to compute timezone for object of class integer;
returning "UTC". This warning will become an error in the next major version of
lubridate.
+ tz(): Don't know how to compute timezone for object of class numeric;
returning "UTC". This warning will become an error in the next major version of
lubridate.
+ Error <expectation_failure>
+ `via_batch <- rlang::eval_tidy(expr,
rlang::new_data_mask(rlang::env(.input = record_batch(tbl))))` threw an
unexpected warning.
+ Message: Expression tz(z) not supported in Arrow; pulling data into R
Review comment:
And, for when you're looking at this tomorrow, check out
https://github.com/apache/arrow/blob/fa699117091917f0992225aff4e8d4c08910162a/r/R/dplyr-eval.R#L18-L60
There are a few bits in there relevant to error messages + how to surface
them to see what's going on as a developer + how you might craft an error
message that will be percolated up
--
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]
