Skip to content

Use random timezone in stateless and stateful tests#15408

Merged
alexey-milovidov merged 67 commits intomasterfrom
random_timezone_for_stateless_tests
Jul 27, 2021
Merged

Use random timezone in stateless and stateful tests#15408
alexey-milovidov merged 67 commits intomasterfrom
random_timezone_for_stateless_tests

Conversation

@alesapin
Copy link
Copy Markdown
Member

@alesapin alesapin commented Sep 28, 2020

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

Changelog category (leave one):

  • Improvement

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Now we ran stateful and stateless tests in random timezones. Fixes #12439. Reading String as DateTime and writing DateTime as String in Protobuf format now respect timezone. Reading UInt16 as DateTime in Arrow and Parquet formats now treat it as Date and then converts to DateTime with respect to DateTime's timezone, because Date is serialized in Arrow and Parquet as UInt16. GraphiteMergeTree now respect time zone for rounding of times. Fixes #5098. Author: @alexey-milovidov.

@robot-clickhouse robot-clickhouse added the pr-build Pull request with build/testing/packaging improvement label Sep 28, 2020
@alexey-milovidov alexey-milovidov self-assigned this Sep 28, 2020
@qoega
Copy link
Copy Markdown
Member

qoega commented Sep 30, 2020

Now we have to fix them :)

@robot-clickhouse robot-clickhouse added pr-improvement Pull request with some product improvements and removed pr-build Pull request with build/testing/packaging improvement labels Jul 24, 2021
@alexey-milovidov
Copy link
Copy Markdown
Member

Finally #5098 is needed.

@alexey-milovidov
Copy link
Copy Markdown
Member

Stress test: #24194 (comment)

@alexey-milovidov
Copy link
Copy Markdown
Member

01098_msgpack_format

Another exception from parallel runs.

@alexey-milovidov alexey-milovidov merged commit d7b2404 into master Jul 27, 2021
@alexey-milovidov alexey-milovidov deleted the random_timezone_for_stateless_tests branch July 27, 2021 13:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-improvement Pull request with some product improvements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Run tests in CI with random time zone. GraphiteMergeTree: use time zone of the data type for rounding.

4 participants