cybertec-postgresql / pgwatch / 31611924710

88%

push

github

[!] use bounded contexts for all database round-trips (#1511)

* - 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
* - add deadline helper and fault-injection test doubles
* - add bounded-deadline fault-injection tests for fetch paths
* - bound fetch and change-detection round-trips by deadline
* - add deadline-bounded tests for Ping and FetchRuntimeInfo
* - make linter happy
* - bound `DbConn.Ping` and `FetchRuntimeInfo` sub-queries
* - extract `DbConn.FetchExtensions`
* - extract `DbConn.FetchControlInfo`
* - extract `DbConn.FetchVersion`
* - add failing test for postgres resolver deadline bound
* - bound `ResolveDatabasesFromPostgres` with a resolver deadline
* - mark criteria as done
* - remove working tasks doc
* - fix warning format

123 of 205 new or added lines in 7 files covered. (60.0%)

5166 of 6032 relevant lines covered (85.64%)

0.98 hits per line