-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
[fix](load) skip cancel rpc if stub is not initialized #33006
Conversation
Thank you for your contribution to Apache Doris. Since 2024-03-18, the Document has been moved to doris-website. |
run buildall |
clang-tidy review says "All clean, LGTM! 👍" |
TPC-H: Total hot run time: 49553 ms
|
TeamCity be ut coverage result: |
TPC-DS: Total hot run time: 200934 ms
|
ClickBench: Total hot run time: 30.78 s
|
Load test result on machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
|
@@ -852,6 +852,10 @@ void VNodeChannel::cancel(const std::string& cancel_msg) { | |||
// But do we need brpc::StartCancel(call_id)? | |||
_cancel_with_msg(cancel_msg); | |||
|
|||
if (_stub == nullptr) { |
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.
_stub is created in VNodeChannel::init, if _stub is null, it will set _is_closed to true.
We should not reach here normally.
If we do not find the root cause, please add some comments and logs here.
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.
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.
I agree with @cambyzju. We should not reach here.
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.
cancel()
could be called when _inited = false
, see #34897 (fixed in that PR)
Proposed changes
Fix coredump in VNodeChannel::cancel
Further comments
If this is a relatively large or complex change, kick off the discussion at [email protected] by explaining why you chose the solution you did and what alternatives you considered, etc...