-
Notifications
You must be signed in to change notification settings - Fork 490
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
remove GrantPriv from AllGlobalPrivs/AllDBPrivs/AllTablePrivs #581
Conversation
LGTM |
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
should we modify |
@AilinKid Yes you're supposed to follow up on the TiDB repository to update the test cases given the latest parser. |
Fixed tests here: |
…p#581) * remove GrantPriv from AllGlobalPrivs/AllDBPrivs/AllTablePrivs * handle testcase error
…p#581) * remove GrantPriv from AllGlobalPrivs/AllDBPrivs/AllTablePrivs * handle testcase error
What problem does this PR solve?
for pingcap/tidb#12330
What is changed and how it works?
remove GrantPriv from AllGlobalPrivs, AllDBPrivs, AllTablePrivs
Check List