ae: make control flow more explicit

7 years ago · b9a8b53d52
parent e88f49e2cc
commit b9a8b53d52
1 changed files with 2 additions and 2 deletions
--- a/agent/ae/ae.go
+++ b/agent/ae/ae.go
@ -142,6 +142,7 @@ FullSync:
 			case <-s.SyncFull.Notif():
 				select {
 				case <-time.After(s.stagger(s.serverUpInterval)):
+					continue FullSync
 				case <-s.ShutdownCh:
 					return
 				}
@ -149,12 +150,11 @@ FullSync:
 			// retry full sync after some time
 			// todo(fs): why don't we use s.Interval here?
 			case <-time.After(s.retryFailInterval + s.stagger(s.retryFailInterval)):
+				continue FullSync

 			case <-s.ShutdownCh:
 				return
 			}
-
-			continue
 		}

 		// do partial syncs until it is time for a full sync again