kaijchen opened a new pull request, #35217:
URL: https://github.com/apache/doris/pull/35217
## Proposed changes
In `~LoadStreamMgr()`, clear load streams before shutdown
SegmentFileWriterThreadPool.
To fix the following check failure.
```
F20240428 03:36:59.966398 10071 threadpool.cpp:253] Check failed: 1 ==
_tokens.size() (1 vs. 97) Threadpool SegmentFileWriterThreadPool destroyed with
97 allocated tokens
*** Check failure stack trace: ***
@ 0x55a0d476b7a6 google::LogMessage::SendToLog()
@ 0x55a0d47681f0 google::LogMessage::Flush()
@ 0x55a0d476bfe9 google::LogMessageFatal::~LogMessageFatal()
@ 0x55a0cad50d19 doris::ThreadPool::~ThreadPool()
@ 0x55a0cac64f8b doris::LoadStreamMgr::~LoadStreamMgr()
@ 0x55a0cac36d02
doris::PInternalServiceImpl::~PInternalServiceImpl()
@ 0x55a0cac372ee
doris::PInternalServiceImpl::~PInternalServiceImpl()
@ 0x55a0d63604f5 brpc::Server::ClearServices()
@ 0x55a0cac2bab1 doris::BRpcService::join()
@ 0x55a0cac2b9cf doris::BRpcService::~BRpcService()
@ 0x55a0c9e3ef7b main
@ 0x7f24519bd083 __libc_start_main
@ 0x55a0c9e3d02a _start
@ (nil) (unknown)
```
## Further comments
If this is a relatively large or complex change, kick off the discussion at
[[email protected]](mailto:[email protected]) by explaining why you
chose the solution you did and what alternatives you considered, etc...
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
