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

fix/TS-3708: check term for role time #22199

Merged
merged 1 commit into from Jul 27, 2023
Merged

fix/TS-3708: check term for role time #22199

merged 1 commit into from Jul 27, 2023

Conversation

LiShunGang
Copy link
Contributor

@LiShunGang LiShunGang commented Jul 26, 2023 •
edited

  1. time of mnode becomes current role in this term
  2. show 0 for offline mnodes

@LiShunGang LiShunGang force-pushed the fix/TS-3708 branch 3 times, most recently from 70f250f to 0b9a262 Compare July 27, 2023 03:33
@LiShunGang
fix: check term for role time
167ecea 
1. time of mnode becomes current role in this term
2. show 0 for offline mnodes
@bgzhao66 bgzhao66 requested review from bgzhao66 and cadem July 27, 2023 07:38
@gccgdb1234 gccgdb1234 merged commit cc3cc73 into 3.0 Jul 27, 2023
1 check passed
@LiShunGang LiShunGang deleted the fix/TS-3708 branch July 27, 2023 07:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gccgdb1234 gccgdb1234 gccgdb1234 approved these changes

@bgzhao66 bgzhao66 bgzhao66 approved these changes

@cadem cadem Awaiting requested review from cadem

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

Successfully merging this pull request may close these issues.

None yet
3 participants
@LiShunGang @gccgdb1234 @bgzhao66