cybertec-postgresql / pgwatch / 31511334613

88%

Build Type

push

github

Committed by web-flow

Commit Message

[*] refactor `reaper.Reap()` (#1505)

* - move `hostRecoveryStatus` into `reaper`
* - extract `reaper.SyncMetricsToSinks()`
* - extract `FilterBySize`, `FilterByMasterOnly`, `TrackRecoveryStatus`
* - refactor `ShutdownOldWorkers` and get rid of `hostsToShutDown`
* - remove redundant arguments
* - remove redundant `srcL` argument
* - add `DbConn.ActiveMetrics()`
* - extract `reaper.FilterSource()`
* - extract `reaper.StartWorker()`
* - rename `hostRecoveryStatus` to `srcRecoveryStatus`
* - get rid of local `logger` var
* - move `prevLoopMonitoredDBs` assignment into `CleanupRemovedWorkers()`
* - make linter happy

Coverage Stats

93 of 120 new or added lines in 3 files covered. (77.5%)

2 existing lines in 1 file now uncovered.

5098 of 5891 relevant lines covered (86.54%)

0.99 hits per line