-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
[enhancement](stmt-forward) make forwarded stmt be able to be killed #31688
[enhancement](stmt-forward) make forwarded stmt be able to be killed #31688
Conversation
Thank you for your contribution to Apache Doris. |
run buildall |
clang-tidy review says "All clean, LGTM! 👍" |
ce1ace0
to
5063df7
Compare
run buildall |
clang-tidy review says "All clean, LGTM! 👍" |
run p0 |
5063df7
to
1c2cc32
Compare
run buildall |
clang-tidy review says "All clean, LGTM! 👍" |
1c2cc32
to
77a08b1
Compare
run buildall |
clang-tidy review says "All clean, LGTM! 👍" |
77a08b1
to
932986a
Compare
run buildall |
TPC-H: Total hot run time: 38025 ms
|
TPC-DS: Total hot run time: 177778 ms
|
ClickBench: Total hot run time: 31.36 s
|
Load test result on machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
|
932986a
to
953e292
Compare
run buildall |
37bb133
to
eb0d4d3
Compare
run buildall |
run feut |
a0a4c0d
resolved conflict |
run buildall |
clang-tidy review says "All clean, LGTM! 👍" |
run buildall |
PR approved by at least one committer and no changes requested. |
clang-tidy review says "All clean, LGTM! 👍" |
run p0 |
run buildall |
clang-tidy review says "All clean, LGTM! 👍" |
TPC-H: Total hot run time: 38734 ms
|
TPC-DS: Total hot run time: 187850 ms
|
Load test result on machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Proposed changes
Forward a cancel request to the corresponding FE when cancelling a forwarded stmt.
Further comments
Test killing by query id.
Test killing by connection.