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.

Sign up for GitHub

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

Jump to bottom

[ISSUE #85] fix preferred leader bug #106

Merged
merged 1 commit into from
Mar 22, 2022
Merged

[ISSUE #85] fix preferred leader bug #106

merged 1 commit into from
Mar 22, 2022

Conversation

Git-Yang
Copy link
Contributor

@Git-Yang Git-Yang commented Jan 26, 2022 •
edited
Loading

#85

  • Add verification to the preferredLeader node to avoid node unavailability caused by ledger falling behind.
  • Added leadershipTransferWaitTimeout. When the ledger cannot catch up within the time threshold, return DLedgerResponseCode.TAKE_LEADERSHIP_FAILED.

@Git-Yang Git-Yang force-pushed the master-preferredLeader branch 2 times, most recently from 081c54d to 8f5ec42 Compare January 26, 2022 07:52
@Git-Yang
Copy link
Contributor Author

@RongtongJin Please help to review the code, thanks!

@RongtongJin RongtongJin merged commit bbaaec6 into openmessaging:master Mar 22, 2022
@RongtongJin
Copy link
Contributor

@RongtongJin Please help to review the code, thanks!

Merged

@cserwen cserwen mentioned this pull request Mar 22, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@RongtongJin RongtongJin RongtongJin approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Git-Yang @RongtongJin