simplify the batcher's timers

nomad/deploymentwatcher/batcher.go

@@ -58,40 +58,28 @@ func (b *EvalBatcher) CreateEval(e *structs.Evaluation) *EvalFuture {
 
 // batcher is the long lived batcher goroutine
 func (b *EvalBatcher) batcher() {
-	timer := time.NewTimer(b.batch)
+	var timerCh <-chan time.Time
 	evals := make(map[string]*structs.Evaluation)
 	for {
 		select {
 		case <-b.ctx.Done():
-			timer.Stop()
 			return
 		case e := <-b.inCh:
-			if len(evals) == 0 {
+			if timerCh == nil {
-				if !timer.Stop() {
+				timerCh = time.After(b.batch)
-					<-timer.C
-				}
-				timer.Reset(b.batch)
 			}
 
 			evals[e.DeploymentID] = e
-		case <-timer.C:
+		case <-timerCh:
-			if len(evals) == 0 {
-				// Reset the timer
-				timer.Reset(b.batch)
-				continue
-			}
-
 			// Capture the future
 			b.l.Lock()
 			f := b.f
 			b.f = nil
 			b.l.Unlock()
 
-			// Shouldn't be possible but protect ourselves
+			// Shouldn't be possible
 			if f == nil {
-				// Reset the timer
+				panic("no future")
-				timer.Reset(b.batch)
-				continue
 			}
 
 			// Capture the evals
@@ -100,14 +88,13 @@ func (b *EvalBatcher) batcher() {
 				all = append(all, e)
 			}
 
-			// Upsert the evals
+			// Upsert the evals in a go routine
-			f.Set(b.raft.UpsertEvals(all))
+			go f.Set(b.raft.UpsertEvals(all))
 
-			// Reset the evals list
+			// Reset the evals list and timer
 			evals = make(map[string]*structs.Evaluation)
+			timerCh = nil
 
-			// Reset the timer
-			timer.Reset(b.batch)
 		}
 	}
 }