-
-
Notifications
You must be signed in to change notification settings - Fork 338
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
Implement database_privileges.get_owner_oid_and_curr_role_db_priv
RPC endpoint
#3760
Conversation
@pavish can you tell me why do we need |
@Anish9901 The owner information is displayed on the UI when checking/modifying permissions and transferring ownership. It is also needed to check if certain functionalities are supported such as dropping the object. The information of the owner i.e. I assume you would like to rename it to |
database_privileges.get_curr_role_priv
RPC endpointdatabase_privileges.get_owner_oid_and_curr_role_db_priv
RPC endpoint
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.
Code looks fine. Once @pavish is happy, I'm happy.
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.
Looks great to me!
Fixes #3735
Checklist
Update index.md
).develop
branch of the repositoryvisible errors.
Developer Certificate of Origin
Developer Certificate of Origin