Closed richfitz closed 3 years ago
Merging #55 (0bfe764) into master (ac82baf) will not change coverage. The diff coverage is
100.00%
.
@@ Coverage Diff @@
## master #55 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 25 25
Lines 1698 1703 +5
=========================================
+ Hits 1698 1703 +5
Impacted Files | Coverage Δ | |
---|---|---|
R/worker_messages.R | 100.00% <100.00%> (ø) |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update ac82baf...0bfe764. Read the comment docs.
This constantly trips (tripped) me up; if you have a worker with a timeout - typical for the cluster - then sending a message does not reset the timer. With this PR most messages do reset the timer except for ones that would make no sense