You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@arrow.apache.org by "thisisnic (via GitHub)" <gi...@apache.org> on 2023/05/17 09:41:49 UTC
[GitHub] [arrow] thisisnic opened a new issue, #35633: [C++] R builds failing with error 'Invalid: Timestamps already have a timezone: 'UTC'. Cannot localize to 'UTC''
thisisnic opened a new issue, #35633:
URL: https://github.com/apache/arrow/issues/35633
### Describe the bug, including details regarding any error messages, version, and platform.
Lots of R builds (e.g. [here](https://github.com/ursacomputing/crossbow/actions/runs/4997978281/jobs/8952919283#step:6:24161)) are failing with the following error:
```
── Error ('test-dplyr-funcs-datetime.R:298:5'): timestamp round trip correctly via strftime and strptime ──
Error in `compute.arrow_dplyr_query(x)`: Invalid: Timestamps already have a timezone: 'UTC'. Cannot localize to 'UTC'.
```
which comes from this line in the C++ library: https://github.com/apache/arrow/blob/b60bb968863f3e6d8bc08cae32b370ea64a3cc22/cpp/src/arrow/compute/kernels/scalar_temporal_unary.cc#L124
This happened since we updated the vendored version of the date library, so perhaps could be related to that?
### Component(s)
C++
--
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: issues-unsubscribe@arrow.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] thisisnic closed issue #35633: [R] R builds failing with error 'Invalid: Timestamps already have a timezone: 'UTC'. Cannot localize to 'UTC''
Posted by "thisisnic (via GitHub)" <gi...@apache.org>.
thisisnic closed issue #35633: [R] R builds failing with error 'Invalid: Timestamps already have a timezone: 'UTC'. Cannot localize to 'UTC''
URL: https://github.com/apache/arrow/issues/35633
--
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: issues-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] jorisvandenbossche commented on issue #35633: [C++] R builds failing with error 'Invalid: Timestamps already have a timezone: 'UTC'. Cannot localize to 'UTC''
Posted by "jorisvandenbossche (via GitHub)" <gi...@apache.org>.
jorisvandenbossche commented on issue #35633:
URL: https://github.com/apache/arrow/issues/35633#issuecomment-1551273829
> This happened since we updated the vendored version of the date library, so perhaps could be related to that?
Was that done in a specific PR? (to understand what changed)
--
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: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] thisisnic commented on issue #35633: [C++] R builds failing with error 'Invalid: Timestamps already have a timezone: 'UTC'. Cannot localize to 'UTC''
Posted by "thisisnic (via GitHub)" <gi...@apache.org>.
thisisnic commented on issue #35633:
URL: https://github.com/apache/arrow/issues/35633#issuecomment-1551281155
Yep, it was in https://github.com/apache/arrow/pull/35612
--
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: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] jorisvandenbossche commented on issue #35633: [C++] R builds failing with error 'Invalid: Timestamps already have a timezone: 'UTC'. Cannot localize to 'UTC''
Posted by "jorisvandenbossche (via GitHub)" <gi...@apache.org>.
jorisvandenbossche commented on issue #35633:
URL: https://github.com/apache/arrow/issues/35633#issuecomment-1551291639
Here:
https://github.com/apache/arrow/blob/99ef3aada1d1589604da8905f3c9100ff1c997e9/r/tests/testthat/test-dplyr-funcs-datetime.R#L292
This adds the format ``"%T%z"``, and that will now actually result in a tz-aware timestamp, while before it didn't parse the timezone correctly.
--
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: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] jorisvandenbossche commented on issue #35633: [C++] R builds failing with error 'Invalid: Timestamps already have a timezone: 'UTC'. Cannot localize to 'UTC''
Posted by "jorisvandenbossche (via GitHub)" <gi...@apache.org>.
jorisvandenbossche commented on issue #35633:
URL: https://github.com/apache/arrow/issues/35633#issuecomment-1551281075
I see that this is https://github.com/apache/arrow/pull/35612. Now, that was actually passing on main, so it might also have been triggered by https://github.com/apache/arrow/pull/35449 (which is failing with this on main, and was actually also already failing on the PR ..)
--
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: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] thisisnic commented on issue #35633: [C++] R builds failing with error 'Invalid: Timestamps already have a timezone: 'UTC'. Cannot localize to 'UTC''
Posted by "thisisnic (via GitHub)" <gi...@apache.org>.
thisisnic commented on issue #35633:
URL: https://github.com/apache/arrow/issues/35633#issuecomment-1551283746
You're right; it is failing on the CI on that PR, e.g. https://github.com/apache/arrow/actions/runs/4900485911/jobs/8751167526
--
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: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] thisisnic commented on issue #35633: [C++] R builds failing with error 'Invalid: Timestamps already have a timezone: 'UTC'. Cannot localize to 'UTC''
Posted by "thisisnic (via GitHub)" <gi...@apache.org>.
thisisnic commented on issue #35633:
URL: https://github.com/apache/arrow/issues/35633#issuecomment-1551231578
This fails on the nightly builds, but I can't replicate this when locally, or when running it on PRs (see https://github.com/apache/arrow/pull/3561).
--
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: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] jorisvandenbossche commented on issue #35633: [C++] R builds failing with error 'Invalid: Timestamps already have a timezone: 'UTC'. Cannot localize to 'UTC''
Posted by "jorisvandenbossche (via GitHub)" <gi...@apache.org>.
jorisvandenbossche commented on issue #35633:
URL: https://github.com/apache/arrow/issues/35633#issuecomment-1551282545
My assumption is that the fix of https://github.com/apache/arrow/pull/35449 caused one of the test dates to become UTC, causing the failure. If that's the case, we will need to fix the test I think
--
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: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org