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

Use the heartbeats shipped with the digest for cluster membership and failure detection #123

Merged
merged 1 commit into from
Feb 19, 2024

Conversation

fulmicoton
Copy link
Contributor

No description provided.

@fulmicoton fulmicoton changed the base branch from main to issue/121-delete-key-gc February 19, 2024 08:17
@fulmicoton fulmicoton force-pushed the leverage-digest2 branch 3 times, most recently from f1fed82 to ac02ca3 Compare February 19, 2024 09:21
@fulmicoton fulmicoton changed the title Leverage digest2 Use the heartbeats shipped with the digest for cluster membership and failure detection Feb 19, 2024
@fulmicoton fulmicoton force-pushed the leverage-digest2 branch 2 times, most recently from 5cf109a to adf1f46 Compare February 19, 2024 09:23
@fulmicoton fulmicoton force-pushed the leverage-digest2 branch 2 times, most recently from 7038731 to f1d16cf Compare February 19, 2024 09:42
chitchat/src/state.rs Outdated Show resolved Hide resolved
@fulmicoton fulmicoton changed the base branch from issue/121-delete-key-gc to main February 19, 2024 15:30
Right now we send the heartbeat of all of the known nodes in the cluster
during digests.

We can use that information for cluster membership.
This will allow nodes to detect new member, join a cluster, and detect
failure considerably faster.

This PR also removes heartbeat from delta message.
Instead it joins the last_gc_version, to fix a bug introduced in #122.

Bugfix:
Avoid updating the time of death of a node when it is redetected as
faulty.

We remove outselves from the list of seeds. Before nodes were regularly gossipping with themselves.

Code improvement:
In the failure detector when we have few samples. We now use additive
smoothing.
@fulmicoton fulmicoton merged commit 7de38f5 into main Feb 19, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants