rabbitmq / ra

A Raft implementation for Erlang and Elixir that strives to be efficient and make it easier to use multiple Raft clusters in a single system.
Other
798 stars 93 forks source link

Handle post_init and unknown terminate crash #404

Closed SimonUnge closed 6 months ago

SimonUnge commented 7 months ago

Proposed Changes

Noticed that if a crash happens in the init callback in a ra_machine, the actual error gets obfuscated by the ra_server_proc not handling a terminate without a state record.

Types of Changes

What types of changes does your code introduce to this project? Put an x in the boxes that apply

Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask on the mailing list. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

Further Comments

If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc.

kjnilsson commented 6 months ago

oops sorry I squashed merged this by mistake when I should have used a merge commit.