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

GRANT OPTION as GRANT should not be described as a regular priv. #4791

Closed
seiya-annie opened this issue Feb 4, 2021 · 3 comments · Fixed by #4799
Closed

GRANT OPTION as GRANT should not be described as a regular priv. #4791

seiya-annie opened this issue Feb 4, 2021 · 3 comments · Fixed by #4799

Comments

@seiya-annie
Copy link

seiya-annie commented Feb 4, 2021

Error Report

This repository is ONLY used to solve issues related to DOCS.
For other issues (related to TiDB, PD, etc), please move to other repositories.

Please answer the following questions before submitting your issue. Thanks!

  1. What is the URL/path of the document related to this issue?

https://docs.pingcap.com/zh/tidb/dev/privilege-management

  1. How would you like to improve it?

delete grand in the table

企业微信截图_e7ea7242-875c-443f-9661-39bec203c1ee

@ti-srebot
Copy link
Contributor

This issue contains Chinese, please use English.

@TomShawn
Copy link
Contributor

TomShawn commented Feb 4, 2021

@seiya-annie Do you mean deleting the Grant row?

@morgo
Copy link
Contributor

morgo commented Feb 4, 2021

The column which says Grant should read Grant Option. But usually the syntax for GRANT option is specified as WITH GRANT OPTION at the end of a GRANT statement, so it's better to just delete the row.

These are also out of date:
image

File was added in pingcap/tidb#19553 and I have manually verified Shutdown, Process, Usage are also implemented correctly. Execute does not apply to TiDB. The only one missing is the index privilege, but it is not so useful. I recommend deleting this section. It looks like index works correctly too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants