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

placement: wrong quorum number #31186

Closed
Tracked by #18030
xhebox opened this issue Dec 30, 2021 · 0 comments · Fixed by #31188
Closed
Tracked by #18030

placement: wrong quorum number #31186

xhebox opened this issue Dec 30, 2021 · 0 comments · Fixed by #31188
Labels
sig/sql-infra SIG: SQL Infra type/enhancement The issue or PR belongs to an enhancement.

Comments

@xhebox
Copy link
Contributor

xhebox commented Dec 30, 2021

Enhancement

Quorum number should be (n/2)+1 but is calculated as ceil(n/2+1) at the time.
@xhebox xhebox added type/enhancement The issue or PR belongs to an enhancement. sig/sql-infra SIG: SQL Infra labels Dec 30, 2021
This was referenced Dec 30, 2021
Closed
Merged
ti-chi-bot pushed a commit that referenced this issue Dec 30, 2021
@xhebox
placement: quorum number done right (#31188)
1cc0b51 
close #31186
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
sig/sql-infra SIG: SQL Infra type/enhancement The issue or PR belongs to an enhancement.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

placement: quorum number done right xhebox/tidb
1 participant
@xhebox