dragosmg commented on a change in pull request #12240:
URL: https://github.com/apache/arrow/pull/12240#discussion_r791766856
##########
File path: r/tests/testthat/test-Array.R
##########
@@ -985,3 +985,14 @@ test_that("Array to C-interface", {
delete_arrow_schema(schema_ptr)
delete_arrow_array(array_ptr)
})
+
+test_that("Array coverts timestamps with missing timezone /assumed local tz
correctly", {
+ withr::with_envvar(c(TZ = "America/Chicago"), {
+ a <- as.POSIXct("1970-01-01 00:00:00")
+ attr(a, "tzone") <- Sys.getenv("TZ")
Review comment:
In my mind there is some inconsistency in the Jira ticket:
> Set the timezone to local time without changing the integer value fo the
timestamp. We store whatever integer R passes to us (21600), with CST as the
timezone set. Display is then "1970-01-01 00:00:00 CST"
If we store the integer that R passes to us (i.e. the _absolute value_ = the
number of seconds from the epoch) then the display will never be `"1970-01-01
00:00:00 CST"` in arrow, but rather the UTC date-time corresponding to the
_absolute value_ with the desired timezone label, which I think it would be
terribly confusing.
--
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]
