[bugfix](becore) local exchange should check receiver's local state to avoid core#45470
[bugfix](becore) local exchange should check receiver's local state to avoid core#45470yiguolei merged 3 commits intoapache:masterfrom
Conversation
|
Thank you for your contribution to Apache Doris. Please clearly describe your PR:
|
|
run buildall |
|
clang-tidy review says "All clean, LGTM! 👍" |
|
TeamCity be ut coverage result: |
|
clang-tidy review says "All clean, LGTM! 👍" |
|
run buildall |
|
TeamCity be ut coverage result: |
|
run buildall |
|
clang-tidy review says "All clean, LGTM! 👍" |
|
TeamCity be ut coverage result: |
|
run buildall |
|
clang-tidy review says "All clean, LGTM! 👍" |
|
TeamCity be ut coverage result: |
|
PR approved by at least one committer and no changes requested. |
|
PR approved by anyone and no changes requested. |
…o avoid core (#45470) _local_recvr depdend on pipeline::ExchangeLocalState* _parent to do some memory counter settings but it only owns a raw pointer, so that the ExchangeLocalState object may be deconstructed. So that I lock the local state to avoid it is deconstruted
…o avoid core (#45470) _local_recvr depdend on pipeline::ExchangeLocalState* _parent to do some memory counter settings but it only owns a raw pointer, so that the ExchangeLocalState object may be deconstructed. So that I lock the local state to avoid it is deconstruted
What problem does this PR solve?
_local_recvr depdend on pipeline::ExchangeLocalState* _parent to do some memory counter settings
but it only owns a raw pointer, so that the ExchangeLocalState object may be deconstructed.
So that I lock the local state to avoid it is deconstruted
Release note
None
Check List (For Author)
Test
Behavior changed:
Does this need documentation?
Check List (For Reviewer who merge this PR)