Mryange opened a new pull request, #50560:
URL: https://github.com/apache/doris/pull/50560
### What problem does this PR solve?
transmit_block should be marked as done when transmitting the last block.
Otherwise, the previous block might reach a memory limit and set done to
null.
```
F20250430 11:49:29.413803 2176884 vdata_stream_recvr.cpp:200] Check failed:
*done != nullptr
*** Check failure stack trace: ***
@ 0x558196be1956 google::LogMessage::SendToLog()
@ 0x558196bde3a0 google::LogMessage::Flush()
@ 0x558196be2199 google::LogMessageFatal::~LogMessageFatal()
@ 0x558193b1659c
doris::vectorized::VDataStreamRecvr::SenderQueue::add_block()
@ 0x558193b1f7b5 doris::vectorized::VDataStreamRecvr::add_block()
@ 0x558193af6cf8 doris::vectorized::VDataStreamMgr::transmit_block()
@ 0x558157f7a685
doris::pipeline::DataStreamRecvrTest_transmit_block_Test::TestBody()
@ 0x5581973c8c0b
testing::internal::HandleSehExceptionsInMethodIfSupported<>()
@ 0x5581973c2a69
testing::internal::HandleExceptionsInMethodIfSupported<>()
@ 0x55819739943a testing::Test::Run()
@ 0x558197399e5e testing::TestInfo::Run()
@ 0x55819739a71e testing::TestSuite::Run()
@ 0x5581973a9dde testing::internal::UnitTestImpl::RunAllTests()
@ 0x5581973c9a56
testing::internal::HandleSehExceptionsInMethodIfSupported<>()
@ 0x5581973c3a61
testing::internal::HandleExceptionsInMethodIfSupported<>()
@ 0x5581973a85d3 testing::UnitTest::Run()
@ 0x5581593d8653 RUN_ALL_TESTS()
```
### Release note
None
### Check List (For Author)
- Test <!-- At least one of them must be included. -->
- [ ] Regression test
- [ ] Unit Test
- [ ] Manual test (add detailed scripts or steps below)
- [ ] No need to test or manual test. Explain why:
- [ ] This is a refactor/code format and no logic has been changed.
- [ ] Previous test can cover this change.
- [ ] No code files have been changed.
- [ ] Other reason <!-- Add your reason? -->
- Behavior changed:
- [ ] No.
- [ ] Yes. <!-- Explain the behavior change -->
- Does this need documentation?
- [ ] No.
- [ ] Yes. <!-- Add document PR link here. eg:
https://github.com/apache/doris-website/pull/1214 -->
### Check List (For Reviewer who merge this PR)
- [ ] Confirm the release note
- [ ] Confirm test cases
- [ ] Confirm document
- [ ] Add branch pick label <!-- Add branch pick label that this PR should
merge into -->
--
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
