Skip to content

Fix some issues with low cardinality and async reads from remote fs#36763

Merged
alesapin merged 2 commits intoClickHouse:masterfrom
kssenii:fix-async-reads
Apr 29, 2022
Merged

Fix some issues with low cardinality and async reads from remote fs#36763
alesapin merged 2 commits intoClickHouse:masterfrom
kssenii:fix-async-reads

Conversation

@kssenii
Copy link
Copy Markdown
Member

@kssenii kssenii commented Apr 28, 2022

Changelog category (leave one):

  • Bug Fix (user-visible misbehaviour in official stable or prestable release)

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):

Fix some issues with async reads from remote filesystem which happened when reading low cardinality.

@robot-ch-test-poll2 robot-ch-test-poll2 added the pr-bugfix Pull request with bugfix, not backported by default label Apr 28, 2022
@alesapin
Copy link
Copy Markdown
Member

Actually, this fix is a hot fix. We need to investigate it and create a proper fix.

@alesapin
Copy link
Copy Markdown
Member

Stress debug -- another issue fixed in #36660

@alesapin
Copy link
Copy Markdown
Member

address Hung check "query": "DROP DATABASE test",

@alesapin
Copy link
Copy Markdown
Member

Thread hung -- 00973_live_view_with_subquery_select_with_aggregation

Copy link
Copy Markdown
Member

@alesapin alesapin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not quite good, but required.

@alesapin alesapin merged commit c72596b into ClickHouse:master Apr 29, 2022
@alexey-milovidov alexey-milovidov added the pr-must-backport Pull request should be backported intentionally. Use this label with great care! label Jul 24, 2022
robot-clickhouse pushed a commit that referenced this pull request Jul 24, 2022
@robot-ch-test-poll robot-ch-test-poll added the pr-backports-created Backport PRs are successfully created, it won't be processed by CI script anymore label Jul 24, 2022
kssenii added a commit that referenced this pull request Aug 1, 2022
Backport #36763 to 22.3: Fix some issues with low cardinality and async reads from remote fs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-backports-created Backport PRs are successfully created, it won't be processed by CI script anymore pr-bugfix Pull request with bugfix, not backported by default pr-must-backport Pull request should be backported intentionally. Use this label with great care!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants