There's no need to set moveRequestCycle and do broadcast twice.

lixiang233 · lixiang233 · commit 5844bc3b9f05 · 2020-04-28T00:43:58.000+08:00
diff --git 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

Original file line number	Diff line number	Diff line change
`@@ -499,8 +499,6 @@ func (p *PriorityQueue) MoveAllToActiveOrBackoffQueue(event string) {`
`499`	`499`	`unschedulablePods = append(unschedulablePods, pInfo)`
`500`	`500`	`}`
`501`	`501`	`p.movePodsToActiveOrBackoffQueue(unschedulablePods, event)`
`502`		`- p.moveRequestCycle = p.schedulingCycle`
`503`		`- p.cond.Broadcast()`
`504`	`502`	`}`
`505`	`503`
`506`	`504`	`// NOTE: this function assumes lock has been acquired in caller`