dragosmg commented on pull request #12357:
URL: https://github.com/apache/arrow/pull/12357#issuecomment-1035525325
Worth noting that the current implementation diverges from `lubridate::tz()`
as indicated in the conversation, namely it does not return `"UTC"` for
strings, numerics or NAs.
``` r
suppressMessages(library(lubridate))
tz(NA)
#> [1] "UTC"
tz(2)
#> Warning: 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.
#> [1] "UTC"
tz(2.2)
#> Warning: 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.
#> [1] "UTC"
tz("string")
#> [1] "UTC"
```
<sup>Created on 2022-02-10 by the [reprex
package](https://reprex.tidyverse.org) (v2.0.1)</sup>
--
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]
