Skip to content

Cancel other processors on query exception#15578

Merged
abyss7 merged 2 commits intoClickHouse:masterfrom
azat:pipeline-executor-cancel-queries-on-exception
Oct 6, 2020
Merged

Cancel other processors on query exception#15578
abyss7 merged 2 commits intoClickHouse:masterfrom
azat:pipeline-executor-cancel-queries-on-exception

Conversation

@azat
Copy link
Copy Markdown
Member

@azat azat commented Oct 3, 2020

Changelog category (leave one):

  • Bug Fix

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Query is finished faster in case of exception. Cancel execution on remote replicas if exception happens.

Detailed description / Documentation draft:
For example this will cancel remote queries on other shards as soon as the query fail on one of them

Cc: @KochetovNicolai

@robot-clickhouse robot-clickhouse added the pr-bugfix Pull request with bugfix, not backported by default label Oct 3, 2020
@azat
Copy link
Copy Markdown
Member Author

azat commented Oct 3, 2020

Performance — 8 errors, 2 too long, 189 unstable

Does not looks related

@alexey-milovidov
Copy link
Copy Markdown
Member

For example this will cancel remote queries on other shards as soon as the query fail on one of them

I thought it was already working this way.

@KochetovNicolai
Copy link
Copy Markdown
Member

I thought it was already working this way.

So did I.

@abyss7 abyss7 self-assigned this Oct 6, 2020
@abyss7 abyss7 merged commit 9594e46 into ClickHouse:master Oct 6, 2020
@azat azat deleted the pipeline-executor-cancel-queries-on-exception branch October 6, 2020 21:48
alesapin pushed a commit to alexelex/ClickHouse that referenced this pull request Oct 8, 2020
* Add a test for canceling distributed queries on error

* Cancel other processors on exception in PipelineExecutor

Fixes: 01514_distributed_cancel_query_on_error
alexey-milovidov added a commit that referenced this pull request Nov 14, 2020
Backport #15578 to 20.9: Cancel other processors on query exception
alexey-milovidov added a commit that referenced this pull request Nov 14, 2020
Backport #15578 to 20.8: Cancel other processors on query exception
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-bugfix Pull request with bugfix, not backported by default

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants