Skip to content

Fix parallel_view_processing#29786

Merged
KochetovNicolai merged 3 commits intoClickHouse:masterfrom
azat:parallel_view_processing-fix
Oct 7, 2021
Merged

Fix parallel_view_processing#29786
KochetovNicolai merged 3 commits intoClickHouse:masterfrom
azat:parallel_view_processing-fix

Conversation

@azat
Copy link
Copy Markdown
Member

@azat azat commented Oct 5, 2021

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

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

Fixes: #28582 (cc @KochetovNicolai )
Fixes: #29621
Closes: #29760

NOTE: marked as not for changelog, since the original PR has not been included to any release yet

azat added 2 commits October 5, 2021 22:20
Number of threads got lost during after QueryPipelineBuilder::addChains()
@robot-clickhouse robot-clickhouse added the pr-not-for-changelog This PR should not be mentioned in the changelog label Oct 5, 2021
@azat azat mentioned this pull request Oct 5, 2021
@KochetovNicolai KochetovNicolai self-assigned this Oct 6, 2021
@KochetovNicolai
Copy link
Copy Markdown
Member

test_dictionaries_dependency seem to be flaky

@KochetovNicolai KochetovNicolai merged commit 4ab2e27 into ClickHouse:master Oct 7, 2021
@azat azat deleted the parallel_view_processing-fix branch October 7, 2021 21:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-not-for-changelog This PR should not be mentioned in the changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Parallel view processing no longer working

4 participants