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
[BugFix] Release workgroup token immediately when fragment is cancelled #19310
[BugFix] Release workgroup token immediately when fragment is cancelled #19310
Conversation
Signed-off-by: ZiheLiu <ziheliu1024@gmail.com>
clang-tidy review says "All clean, LGTM! 👍" |
Kudos, SonarCloud Quality Gate passed! |
run starrocks_be_unittest |
1 similar comment
run starrocks_be_unittest |
run starrocks_be_unittest |
run starrocks_admit_test |
clang-tidy review says "All clean, LGTM! 👍" |
run starrocks_fe_unittest |
2 similar comments
run starrocks_fe_unittest |
run starrocks_fe_unittest |
[FE PR Coverage Check]😞 fail : 19 / 28 (67.86%) file detail
|
@Mergifyio backport branch-3.0 |
@Mergifyio backport branch-2.4 |
✅ Backports have been created
|
@Mergifyio backport branch-2.3 |
✅ Backports have been created
|
✅ Backports have been created
|
✅ Backports have been created
|
…ed (#19310) Signed-off-by: ZiheLiu <ziheliu1024@gmail.com> (cherry picked from commit 6f0bd88) # Conflicts: # be/src/exec/pipeline/fragment_context.cpp # be/src/exec/pipeline/query_context.cpp # be/src/exec/pipeline/query_context.h # be/src/exec/pipeline/scan/scan_operator.cpp # fe/fe-core/src/main/java/com/starrocks/qe/Coordinator.java
clang-tidy review says "All clean, LGTM! 👍" |
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
…ed (StarRocks#19310) Signed-off-by: ZiheLiu <ziheliu1024@gmail.com>
…ed (StarRocks#19310) Signed-off-by: ZiheLiu <ziheliu1024@gmail.com>
…ed (StarRocks#19310) Signed-off-by: ZiheLiu <ziheliu1024@gmail.com>
…ed (StarRocks#19310) Signed-off-by: ZiheLiu <ziheliu1024@gmail.com>
What type of PR is this:
Which issues of this PR fixes :
Fixes #
TODO.
Problem Summary(Required) :
Otherwise, it can only be released after query_delivery_timeout (5min by default) sometimes.
Otherwise, the cancellation RPC may arrive at BE before the delivery fragment instance RPC, causing the instances and drivers to become stale and only able to be released after query timeout.
Checklist:
Bugfix cherry-pick branch check: