Skip to content
This repository has been archived by the owner on Jun 20, 2024. It is now read-only.

own action to reclaim when peer does not exist #3416

Merged
merged 2 commits into from
Oct 5, 2018
Merged

own action to reclaim when peer does not exist #3416

merged 2 commits into from
Oct 5, 2018

Conversation

murali-reddy
Copy link
Contributor

@murali-reddy murali-reddy commented Sep 27, 2018

handles an edge case where peer claimed to own the action to reclaim but no longer exists hence lock persists forever, fix makes a peer own the reclaim when nonexistent node found

Fixes #3386

…) but no longer

exists hence lock persists foever, fix makes a peer own the reclaim when nonexistent node found

Fixes #3386
Copy link
Contributor

@bboreham bboreham left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Some thoughts below.

} else {
// handle an edge case where peer claimed to own the action to reclaim but no longer
// exists hence lock persists foever
peerExists := false

This comment was marked as abuse.

This comment was marked as abuse.

break
}
}
nonExistentPeer = !peerExists

This comment was marked as abuse.

@bboreham bboreham merged commit 078b76f into master Oct 5, 2018
@bboreham bboreham added this to the 2.5 milestone Oct 9, 2018
@bboreham bboreham deleted the issue-3386 branch December 24, 2018 12:03
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Not removing unreachable peers due to lock from nonexistent peer
2 participants