-
Notifications
You must be signed in to change notification settings - Fork 22
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
Add the support of GRANT PRIVILEGE #24
Conversation
Pull Request Test Coverage Report for Build 6743550597
💛 - Coveralls |
@Lance726 I don't add enough test sql tests since cannot find more cases, may add more later. |
ok~ |
fb790d2
to
228b39d
Compare
@orginux Can have a try if you like. |
I would suggest adding the GRANT ALL ON *.* TO admin_role WITH GRANT OPTION; |
Thank you, done in 57ca48c |
Great, thanks for the quick response and solving. GRANT SELECT ON database.table_1 TO table_1_select_role;
GRANT INSERT ON database.table_2 TO table_2_insert_role; but currently, the parser supports only grants with parameters. |
It should also be possible to submit multiple grants in a single query: GRANT SELECT, dictGet ON *.* TO select_all_role; |
@orginux Yes, those are missing keywords in GRANT PRIVILEGE and now have been fixed. |
This closes #18