luxolus
/
open-consul
2
0
Fork 0
Code Issues 1 Pull Requests Packages Releases Wiki Activity

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

This commit is contained in:
Armon Dadgar 2015-10-12 20:30:11 -07:00
parent 36a11af084
commit 0b77960349
1 changed files with 8 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
command/agent/local.go
View File

@ -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}
}
}