-
Notifications
You must be signed in to change notification settings - Fork 688
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
reference/best-practices: add PD scheduling best practice #1655
Conversation
0fed312
to
b1fa73e
Compare
15d18f4
to
f014071
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
first batch of review
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
first batch of review
pingcap/docs-cn#2106 removed two extra spaces from |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Please apply to other versions.
c54db32
to
e53e799
Compare
What is changed, added or deleted?
This PR
What is the related PR or file link(s)?
pingcap/docs-cn#1990
pingcap/docs-cn#2003
Which version does your change affect?
dev, v2.1, v3.0 and v3.1