Merge pull request #1296 from hashicorp/f-fast-sync

agent: remove an N^2 check. See #1265

command/agent/local.go

@@ -389,16 +389,15 @@ func (l *localState) setSyncState() error {
 		l.serviceStatus[id] = syncStatus{inSync: equal}
 	}
 
-	for id, _ := range l.checks {
-		// Sync any check which doesn't exist on the remote side
-		found := false
+	// Index the remote health checks to improve efficiency
+	checkIndex := make(map[string]*structs.HealthCheck, len(checks))
 	for _, check := range checks {
-			if check.CheckID == id {
-				found = true
-				break
-			}
+		checkIndex[check.CheckID] = check
 	}
-		if !found {
+
+	// Sync any check which doesn't exist on the remote side
+	for id, _ := range l.checks {
+		if _, ok := checkIndex[id]; !ok {
 			l.checkStatus[id] = syncStatus{inSync: false}
 		}
 	}