diff --git a/be/src/vec/exec/scan/scanner_scheduler.cpp b/be/src/vec/exec/scan/scanner_scheduler.cpp index 4223bd7abb..6fd49aa9d1 100644 --- a/be/src/vec/exec/scan/scanner_scheduler.cpp +++ b/be/src/vec/exec/scan/scanner_scheduler.cpp @@ -213,6 +213,8 @@ void ScannerScheduler::_schedule_scanners(std::shared_ptr ctx) { while (iter != this_run.end()) { std::shared_ptr scanner_delegate = (*iter).lock(); if (scanner_delegate == nullptr) { + // Has to ++, or there is a dead loop + iter++; continue; } scanner_delegate->_scanner->start_wait_worker_timer(); @@ -220,7 +222,7 @@ void ScannerScheduler::_schedule_scanners(std::shared_ptr ctx) { this->_scanner_scan(this, ctx, scanner_ref); }); if (s.ok()) { - this_run.erase(iter++); + iter++; } else { ctx->set_status_on_error(s); break; @@ -230,6 +232,8 @@ void ScannerScheduler::_schedule_scanners(std::shared_ptr ctx) { while (iter != this_run.end()) { std::shared_ptr scanner_delegate = (*iter).lock(); if (scanner_delegate == nullptr) { + // Has to ++, or there is a dead loop + iter++; continue; } scanner_delegate->_scanner->start_wait_worker_timer(); @@ -259,7 +263,7 @@ void ScannerScheduler::_schedule_scanners(std::shared_ptr ctx) { ret = _remote_scan_thread_pool->offer(task); } if (ret) { - this_run.erase(iter++); + iter++; } else { ctx->set_status_on_error( Status::InternalError("failed to submit scanner to scanner pool"));