Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[R] Issue with tzdb 0.4.0 and the shipped arrow tz.cpp #35594

Closed
DavisVaughan opened this issue May 15, 2023 · 1 comment · Fixed by #35612
Closed

[R] Issue with tzdb 0.4.0 and the shipped arrow tz.cpp #35594

DavisVaughan opened this issue May 15, 2023 · 1 comment · Fixed by #35612
Assignees
Milestone

Comments

@DavisVaughan
Copy link
Contributor

Describe the bug, including details regarding any error messages, version, and platform.

I released tzdb 0.4.0 last week and got this report:
r-lib/tzdb#33

I also see this on arrow's CRAN windows builds:
https://cran.r-project.org/web/checks/check_results_arrow.html

I was able to reproduce this:

  • With CRAN arrow
  • With tzdb 0.4.0
  • On Windows
> library(arrow)
		#STDOFF	2:05:08.9
		#STDOFF	8:05:43.2
		#STDOFF	7:36:41.7
		#STDOFF	-0:25:21.1
		#STDOFF	1:39:49.2
		#STDOFF	-0:36:44.68
		#STDOFF	-4:56:01.6
		#STDOFF	-3:58:29.2
		#STDOFF	-4:19:18.3
		#STDOFF	-5:36:13.3
		#STDOFF	-4:56:16.4
		#STDOFF	2:05:08.9
		#STDOFF	8:05:43.2
		#STDOFF	7:36:41.7
		#STDOFF	-0:25:21.1
		#STDOFF	1:39:49.2
		#STDOFF	-0:36:44.68
		#STDOFF	-4:56:01.6
		#STDOFF	-3:58:29.2
		#STDOFF	-4:19:18.3
		#STDOFF	-5:36:13.3
		#STDOFF	-4:56:16.4

The issue is that the time zone database (https://github.com/eggert/tz) that gets updated regularly had an update that was incompatible with the <date> library's tz.cpp file. Howard patched it quickly here (HowardHinnant/date@22ceabf) but it means that if you use the text tzdata at or newer than 2022b then you'll see these messages until you update tz.cpp.

tzdb 0.4.0 included two things:

  • An update of the tzdata to 2023c
  • An update of the date files, i.e. tz.cpp

Because both were updated at once, I didn't see any incompatibility issues and anyone who links to tzdb won't either (like clock and readr and vroom).

However, it seems like arrow uses:

Because tz.cpp isn't updated, when people update to tzdb 0.4.0 they start seeing these messages.

I think the solution is to just update your vendored copy of the date library files, but I'm not completely sure. If you can reproduce on your own Windows machines, then maybe try updating your vendored copy of date and see if the messages go away?

Component(s)

R

@thisisnic
Copy link
Member

Thanks for reporting this @DavisVaughan and for the suggestions on how to fix; will take a look.

@pitrou pitrou added this to the 13.0.0 milestone May 16, 2023
pitrou pushed a commit that referenced this issue May 16, 2023
There was an issue with tzdb 0.4.0 and the shipped arrow `tz.cpp`.

This PR bumps the vendored version of the date library to commit `cc4685a21e4a4fdae707ad1233c61bbaff241f93`.
* Closes: #35594

Authored-by: Nic Crane <thisisnic@gmail.com>
Signed-off-by: Antoine Pitrou <antoine@python.org>
@raulcd raulcd modified the milestones: 13.0.0, 12.0.1 May 16, 2023
rtpsw pushed a commit to rtpsw/arrow that referenced this issue May 16, 2023
There was an issue with tzdb 0.4.0 and the shipped arrow `tz.cpp`.

This PR bumps the vendored version of the date library to commit `cc4685a21e4a4fdae707ad1233c61bbaff241f93`.
* Closes: apache#35594

Authored-by: Nic Crane <thisisnic@gmail.com>
Signed-off-by: Antoine Pitrou <antoine@python.org>
thisisnic added a commit to thisisnic/arrow that referenced this issue May 18, 2023
There was an issue with tzdb 0.4.0 and the shipped arrow `tz.cpp`.

This PR bumps the vendored version of the date library to commit `cc4685a21e4a4fdae707ad1233c61bbaff241f93`.
* Closes: apache#35594

Authored-by: Nic Crane <thisisnic@gmail.com>
Signed-off-by: Antoine Pitrou <antoine@python.org>
raulcd pushed a commit that referenced this issue May 30, 2023
There was an issue with tzdb 0.4.0 and the shipped arrow `tz.cpp`.

This PR bumps the vendored version of the date library to commit `cc4685a21e4a4fdae707ad1233c61bbaff241f93`.
* Closes: #35594

Authored-by: Nic Crane <thisisnic@gmail.com>
Signed-off-by: Antoine Pitrou <antoine@python.org>
klevdiamanti added a commit to Olink-Proteomics/OlinkRPackage that referenced this issue Nov 22, 2023
…ean_R4.3

Test CI with R 4.3.0

Windows CI failing because of known (apache/arrow#35594) issues between arrow and tzdb
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants