vagetablechicken edited a comment on issue #3416: URL: https://github.com/apache/incubator-doris/issues/3416#issuecomment-626102191
Current fragment_mgr use https://github.com/apache/incubator-doris/blob/0430714ca9b0d910ba9b290276da05adfc752f8f/be/src/runtime/fragment_mgr.cpp#L461-L463 to avoid ThreadPool::offer() failed. If we replace it with kudu thread pool, we'll met offer() failed. So we must handle this situation: ```exec_state created -> prepare -> submit to thread pool failed (means never execute)-> destory``` FragmentMgr's thread pool is `PriorityThreadPool`, it use blocking_put() to offer queue. ### So we need replace it with thread pool first. ### Then add a UT to test if offer failed. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
