zclllyybb opened a new pull request, #19331: URL: https://github.com/apache/doris/pull/19331
# Proposed changes ``` F0505 22:46:41.875746 497986 vdata_stream_recvr.cpp:87] Check failed: _num_remaining_senders == 0 (26 vs. 0) *** Check failure stack trace: *** @ 0x55c8198dc95d google::LogMessage::Fail() @ 0x55c8198dee99 google::LogMessage::SendToLog() @ 0x55c8198dc4c6 google::LogMessage::Flush() @ 0x55c8198df509 google::LogMessageFatal::~LogMessageFatal() @ 0x55c81814cccb doris::vectorized::VDataStreamRecvr::SenderQueue::_inner_get_batch() @ 0x55c81815fa74 doris::vectorized::VDataStreamRecvr::PipSenderQueue::get_batch() @ 0x55c818155072 doris::vectorized::VDataStreamRecvr::get_next() @ 0x55c808360620 doris::vectorized::VExchangeNode::get_next() @ 0x55c80071a114 doris::ExecNode::pull() ``` ## Problem summary the check of emptiness should be locked in the same lock of real use to ensure correctness. usage before is defective. now fixed it. ## Checklist(Required) * [ ] Does it affect the original behavior * [ ] Has unit tests been added * [ ] Has document been added or modified * [ ] Does it need to update dependencies * [x] Is this PR support rollback (If NO, please explain WHY) ## Further comments the lock may could be split for the usage of the specific object. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org For additional commands, e-mail: commits-h...@doris.apache.org