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

logs get littered with bogus warning about "unrecognized timezone" #181

Closed
pkit opened this issue May 1, 2023 · 2 comments · Fixed by #185
Closed

logs get littered with bogus warning about "unrecognized timezone" #181

pkit opened this issue May 1, 2023 · 2 comments · Fixed by #185
Assignees
Labels
bug Something isn't working minor

Comments

@pkit
Copy link

pkit commented May 1, 2023

Describe the bug

WARNING  clickhouse_connect.driver.client:client.py:124 Unrecognized timezone None received from ClickHouse

Any query that doesn't explicitly set QueryResultDetails in the clickhouse-server will return with no details.
I.e. no headers like X-ClickHouse-Format, X-ClickHouse-Timezone, X-ClickHouse-Query-Id would be present.

Steps to reproduce

  1. INSERT INTO table1 VALUES (...)
  2. See the warning pop up.

Expected behaviour

No warning, as some queries are expected not to have X-ClickHouse-Timezone in response

Version

Checked in the current clickhouse-connect/main vs ClickHouse/master

@pkit pkit added the bug Something isn't working label May 1, 2023
@genzgd genzgd self-assigned this May 1, 2023
@genzgd genzgd added the minor label May 1, 2023
@genzgd
Copy link
Collaborator

genzgd commented May 2, 2023

Thanks for the report! This should be fixed in the 0.5.23 version which will be released shortly.

@genzgd genzgd linked a pull request May 2, 2023 that will close this issue
2 tasks
@pkit
Copy link
Author

pkit commented May 3, 2023

Nice!

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

Successfully merging a pull request may close this issue.

2 participants