Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

khepri_machine: Handle shutdown error from ra:process_command/3 #211

Merged
merged 1 commit into from
Jul 20, 2023

Conversation

the-mikedavis
Copy link
Member

ra:process_command/3 can now return {error,shutdown} if the server shuts down before it replies to the caller (see here). We can handle this the same way as the other errors: find a new leader and retry.

`ra:process_command/3` can now return `{error,shutdown}` if the server
shuts down before it replies to the caller. We can handle this the same
way as the other errors: find a new leader and retry.
@the-mikedavis the-mikedavis self-assigned this Jul 18, 2023
@codecov
Copy link

codecov bot commented Jul 18, 2023

Codecov Report

Patch coverage has no change and project coverage change: +0.03 🎉

Comparison is base (a231525) 89.36% compared to head (812bb82) 89.39%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #211      +/-   ##
==========================================
+ Coverage   89.36%   89.39%   +0.03%     
==========================================
  Files          20       20              
  Lines        2820     2820              
==========================================
+ Hits         2520     2521       +1     
+ Misses        300      299       -1     
Flag Coverage Δ
erlang-24 88.26% <ø> (ø)
erlang-25 88.29% <ø> (+0.03%) ⬆️
erlang-26 89.04% <ø> (+0.21%) ⬆️
os-ubuntu-latest 89.39% <ø> (+0.17%) ⬆️
os-windows-latest 88.26% <ø> (+0.14%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/khepri_machine.erl 95.55% <ø> (ø)

... and 1 file with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@dumbbell dumbbell merged commit 23c1305 into main Jul 20, 2023
10 checks passed
@dumbbell dumbbell deleted the md-ra-error-shutdown-response branch July 20, 2023 18:35
@dumbbell dumbbell added this to the v0.8.0 milestone Jul 26, 2023
@dumbbell dumbbell added the enhancement New feature or request label Jul 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants