[Heartbeat] Clean up some nits from Explicit IDs (9697) (#9926)

This cleans up a couple small points missed in #9697
elastic · Jan 7, 2019 · 05bd95f · 05bd95f
1 parent a3b28de
commit 05bd95f
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/heartbeat/monitors/monitor.go b/heartbeat/monitors/monitor.go
@@ -309,7 +309,7 @@ func (m *Monitor) Stop() {
 	defer m.internalsMtx.Unlock()
 
 	// Free up the monitor ID for reuse
-	uniqueMonitorIDs.Delete(m.id)
+	defer uniqueMonitorIDs.Delete(m.id)
 
 	for _, t := range m.configuredJobs {
 		t.Stop()
@@ -329,7 +329,8 @@ func WithMonitorMeta(id string, name string, typ string, origJob Job) Job {
 			"monitor": common.MapStr{
 				"id":   id,
 				"name": name,
-				"type": typ},
+				"type": typ,
+			},
 		},
 		origJob,
 	)