Skip to content

Conversation

MrCroxx
Copy link
Member

@MrCroxx MrCroxx commented Jul 4, 2023

Use Store::shutdown_runners() to gracefully shutdown runners.`

Close #44 .

Use `Store::shutdown_runners()` to gracefully shutdown runners.`

Signed-off-by: MrCroxx <[email protected]>
@MrCroxx MrCroxx added the bug Something isn't working label Jul 4, 2023
@MrCroxx MrCroxx self-assigned this Jul 4, 2023
@codecov-commenter
Copy link

codecov-commenter commented Jul 4, 2023

Codecov Report

Merging #47 (d7362f9) into main (f067d1c) will decrease coverage by 0.85%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main      #47      +/-   ##
==========================================
- Coverage   42.05%   41.20%   -0.85%     
==========================================
  Files          33       30       -3     
  Lines        3365     3422      +57     
==========================================
- Hits         1415     1410       -5     
- Misses       1950     2012      +62     
Impacted Files Coverage Δ
foyer-storage-bench/src/main.rs 0.00% <0.00%> (-0.32%) ⬇️
foyer-storage/src/flusher.rs 0.00% <0.00%> (ø)
foyer-storage/src/reclaimer.rs 0.00% <0.00%> (ø)
foyer-storage/src/region_manager.rs 0.00% <0.00%> (ø)
foyer-storage/src/store.rs 0.00% <0.00%> (ø)

... and 4 files with indirect coverage changes

@MrCroxx MrCroxx merged commit 8979084 into main Jul 4, 2023
@MrCroxx MrCroxx deleted the xx/runner-gracefully-shutdown branch July 4, 2023 09:08
MrCroxx added a commit that referenced this pull request Apr 17, 2024
* fix: gracefullly shutdown runners

Use `Store::shutdown_runners()` to gracefully shutdown runners.`

Signed-off-by: MrCroxx <[email protected]>

* log warn if send failed instead of panic

Signed-off-by: MrCroxx <[email protected]>

---------

Signed-off-by: MrCroxx <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

bug: flusher & reclaimer needs graceful shutdown

2 participants