-
Notifications
You must be signed in to change notification settings - Fork 752
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
feat(planner): migrate revoke to planner v2 #6066
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Ignored Deployment
|
Thanks for the contribution! Please review the labels and make any necessary changes. |
f49767d
to
d9730ae
Compare
SHOW GRANTS FOR 'test-user'; | ||
REVOKE SELECT ON b.* FROM ROLE 'test'; | ||
SHOW GRANTS FOR 'test-user'; | ||
SHOW GRANTS FOR ROLE 'test'; |
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.
BTW, I think this test has some questions. But it no matters with this pr. So I will create a new issue to disscuss about it.
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.
19ce575
to
f2706ae
Compare
f2706ae
to
bd2e8ec
Compare
@BohuTANG It looks like the databend-query not start.
|
bd2e8ec
to
149f368
Compare
I hereby agree to the terms of the CLA available at: https://databend.rs/dev/policies/cla/
Summary
migrate revoke to planner v2
the issue Migrate
USER
related DDL statements to new planner #5777 containuser ddl
,role
,grant
,revoke
I think all of them is account manager. So refactor the file struct.Changelog
Related Issues
Fixes #5777