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

Windows timezone download fails harlequin, version 1.24.1 on windows 11 #652

Closed
4 of 12 tasks
paulrobello opened this issue Sep 29, 2024 · 1 comment · Fixed by #661
Closed
4 of 12 tasks

Windows timezone download fails harlequin, version 1.24.1 on windows 11 #652

paulrobello opened this issue Sep 29, 2024 · 1 comment · Fixed by #661
Labels
bug Something isn't working upstream

Comments

@paulrobello
Copy link

Before Proceeding, please acknowledge:

  • I have consulted the Troubleshooting guide.
  • I have searched Issues and Discussions in this repo.
  • Feature requests should be initiated as Discussions. This is a bug report.

Describe the bug
windows 1st fails to download / use timezone database

To Reproduce
run harlequin without any params

Expected behavior
App should download tz db and start up

Actual behavior
Application raises the following error
Rule Mexico 1931 only - April 30 0:00 1:00 D
Mexico 1931 1931 Jan/01 00:00:00 00:00:00
╭─ Harlequin Timezone Support ──────────────────────────────────────────────╮
│ Harlequin could not find a timezone database, which it needs to support Apache Arrow's timestamptz features. It is downloading one now (it │
│ will only do this once). For more info, see https://harlequin.sh/docs/troubleshooting/timezone-windows
╰──────────────────────────────────────────────────────────────────╯
Rule Mexico 1931 only - April 30 0:00 1:00 D
Mexico 1931 1931 Jan/01 00:00:00 00:00:00
╭─ Harlequin Timezone Error ───────────────────────────────────────────────────╮
│ Harlequin was not able to download a timezone database. Without a timezone database, Harlequin may crash if you attempt to load timestamptz │
│ values into the results viewer. To use Harlequin anyway, set the --no-download-tzdata option. │
│ For more info, see https://harlequin.sh/docs/troubleshooting/timezone-windows
│ Download failed with the following exception: │
│ oops: bad dow name: il │
╰─────────────────────────────────────────────────────────────────────╯
Contributing
Are you interested in contributing a fix?

  • Yes
  • Maybe
  • [ x] No

What is the output of harlequin --version?
harlequin, version 1.24.1

Installed Adapters:

  • duckdb, version 1.24.1
  • sqlite, version 1.24.1
  • postgres, version 0.4.0

What database adapter are you using with Harlequin? (Default is duckdb)
duckdb and postgres

What other options are you using when invoking Harlequin? (If you are using a profile, please add relevant items from your profile here).

No extra options

Can you tell us more about your system?

  • Shell: Bash
  • Terminal: Windows Terminal Preview
  • OS of the shell: Windows 11 Pro
  • Default locale: en_US/UTF-8

How did you install Harlequin?

  • pipx
  • conda/mamba
  • pip (global install)
  • pip + venv/virtualenv or pipenv
  • Poetry
  • uv/Rye
  • other: _____________
@tconbeer tconbeer added the bug Something isn't working label Sep 30, 2024
@tconbeer
Copy link
Owner

tconbeer commented Sep 30, 2024

Thanks for the report; this was mentioned in a PR last week, and there is some more background here: #646 (comment)

The tl;dr is that the lastest TZ database contains an error.

The current workarounds are:

  1. Run harlequin with the --no-download-tzdata option (with the caveats in the error message)
  2. Manually download and extract an IANA TZ database that is NOT 2024b. Harlequin checks the following locations for such a database: %USERPROFILE%\Downloads\tzdata, %USERPROFILE%\AppData\Local\harlequin\harlequin\tzdata
  3. Manually edit the downloaded file at %USERPROFILE%\AppData\Local\harlequin\harlequin\tzdata\northamerica and replace all instances of the word April with Apr (there is only 1 entry in the actual code, but there are many instance in comments that can be replaced safely)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working upstream
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants