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.

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

toc, reference: delete show create user usage from v2.1 #2000

Merged
merged 5 commits into from
Nov 12, 2019

Conversation

aytrack
Copy link
Contributor

@aytrack aytrack commented Oct 31, 2019

What is changed, added or deleted?

  • delete v2.1/reference/sql/statements/show-create-user.md page
  • remove all link to show create user in v2.1
  • replace all show create user sql with SELECT USER, HOST, PASSWORD FROM mysql.user WHERE USER ="" in v2.1

What is the related PR or file link(s)?

pingcap/tidb#9240
https://pingcap.com/docs-cn/dev/releases/3.0.0-beta.1/

TiDB v2.1 does not support the show create user SQL statement, so I delete them from the docs.

Which version does your change affect?

v2.1

@CLAassistant
Copy link

CLAassistant commented Oct 31, 2019

CLA assistant check
All committers have signed the CLA.

@lilin90 lilin90 changed the title Delete show create user usage from v2.1 toc, reference: delete show create user usage from v2.1 Oct 31, 2019
@lilin90 lilin90 requested a review from jackysp October 31, 2019 07:39
Copy link
Member

@lilin90 lilin90 left a comment

Choose a reason for hiding this comment

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

LGTM

@lilin90 lilin90 added the translation/doing This PR’s assignee is translating this PR. label Oct 31, 2019
@lilin90 lilin90 self-assigned this Oct 31, 2019
@yikeke yikeke removed their request for review November 1, 2019 05:47
@lilin90
Copy link
Member

lilin90 commented Nov 11, 2019

@aytrack When the content is ready, please also update v3.1 as well, thanks~

@TomShawn This one does not apply to v3.1. Please check the PR description and changes.

@lilin90 lilin90 added the v2.1 label Nov 11, 2019
@lilin90
Copy link
Member

lilin90 commented Nov 11, 2019

@jackysp PTAL~ Thanks!

Copy link
Member

@jackysp jackysp left a comment

Choose a reason for hiding this comment

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

LGTM

@jackysp jackysp merged commit de8e111 into pingcap:master Nov 12, 2019
@lilin90 lilin90 added translation/done This PR has been translated from English into Chinese and updated to pingcap/docs-cn in a PR. and removed translation/doing This PR’s assignee is translating this PR. labels Nov 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
translation/done This PR has been translated from English into Chinese and updated to pingcap/docs-cn in a PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants