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.

Sign up for GitHub

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

Jump to bottom

feat: add owner field for database/schema/table/source #2814

Closed
yezizp2012 opened this issue May 25, 2022 · 2 comments · Fixed by #3205
Closed

feat: add owner field for database/schema/table/source #2814

yezizp2012 opened this issue May 25, 2022 · 2 comments · Fixed by #3205
Assignees
@cnissnzg
Labels
component/frontend Protocol, parsing, binder. component/meta Meta related issue. good first issue Good for newcomers help wanted Issues that need help from contributors type/enhancement Improvements to existing implementation.

Comments

@yezizp2012
Copy link
Contributor

yezizp2012 commented May 25, 2022 •
edited by hengm3467

Requirements
We want to support the owner field for database / schema / table / source objects. The implementation should be based on the user account management design recorded in #2193 , and should work as an addition to the feature chunk for supporting the concepts of and operations related to database, schema, and users (tracked in #2150 )
References
Please use #2150 as the main reference
@yezizp2012 yezizp2012 added type/enhancement Improvements to existing implementation. good first issue Good for newcomers help wanted Issues that need help from contributors component/meta Meta related issue. component/frontend Protocol, parsing, binder. labels May 25, 2022
@liurenjie1024
Copy link
Contributor

It seems to me that database/schema/table/source <-> owner should be n:n mapping?

@yezizp2012
Copy link
Contributor Author

yezizp2012 commented May 25, 2022 •
edited

Each database(others as well) can only have one owner. But we can have more than one superuser, or grant permissions specifically to others by owner or superuser, which is compatible with PostgreSQL. In the future, we can also introduce transforming ownership like PostgreSQL does: ALTER DATABASE name OWNER TO new_owner;

This was referenced Jun 8, 2022
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@cnissnzg cnissnzg

Labels
component/frontend Protocol, parsing, binder. component/meta Meta related issue. good first issue Good for newcomers help wanted Issues that need help from contributors type/enhancement Improvements to existing implementation.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat(frontend): add owner field for database/schema/table/source risingwavelabs/risingwave
3 participants
@liurenjie1024 @yezizp2012 @cnissnzg