Unverified Commit df4cd191 authored by Yan Ru Pei's avatar Yan Ru Pei Committed by GitHub
Browse files

chore: remove dead AllWorkersBusy handling in KvScheduler (#5869)


Signed-off-by: default avatarPeaBrane <yanrpei@gmail.com>
parent 8dd1fc07
......@@ -44,9 +44,6 @@ pub enum KvSchedulerError {
#[error("no endpoints available to route work")]
NoEndpoints,
#[error("all workers busy")]
AllWorkersBusy,
#[error("endpoint subscriber shutdown")]
SubscriberShutdown,
......@@ -262,12 +259,6 @@ impl KvScheduler {
tokio::time::sleep(Duration::from_millis(5)).await;
continue;
}
// TODO: this is not actually hooked up
Err(KvSchedulerError::AllWorkersBusy) => {
tracing::trace!("all workers busy; waiting for more capacity");
tokio::time::sleep(Duration::from_millis(5)).await;
continue;
}
Err(e) => {
tracing::error!("error scheduling request: {:?}", e);
break;
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment