From 5844bc3b9f055dc006ff1bf128ba4037ded93cff Mon Sep 17 00:00:00 2001 From: lixiang Date: Tue, 28 Apr 2020 00:43:58 +0800 Subject: [PATCH] There's no need to set moveRequestCycle and do broadcast twice. --- pkg/scheduler/internal/queue/scheduling_queue.go | 2 -- 1 file changed, 2 deletions(-) diff --git a/pkg/scheduler/internal/queue/scheduling_queue.go b/pkg/scheduler/internal/queue/scheduling_queue.go index 2311625a841..8359feb40bf 100644 --- a/pkg/scheduler/internal/queue/scheduling_queue.go +++ b/pkg/scheduler/internal/queue/scheduling_queue.go @@ -499,8 +499,6 @@ func (p *PriorityQueue) MoveAllToActiveOrBackoffQueue(event string) { unschedulablePods = append(unschedulablePods, pInfo) } p.movePodsToActiveOrBackoffQueue(unschedulablePods, event) - p.moveRequestCycle = p.schedulingCycle - p.cond.Broadcast() } // NOTE: this function assumes lock has been acquired in caller