shaofengshi commented on code in PR #5293: URL: https://github.com/apache/gravitino/pull/5293#discussion_r1838267044
########## docs/cli.md: ########## @@ -503,3 +503,24 @@ gcli tag update --tag tagA --rename newTag ```bash gcli tag update --tag tagA --comment "new comment" ``` + +### Owners commands + +#### List an owner + +```bash +gcli catalog details --name postgres --owner +``` + +#### Set an owner to a user + +```bash +gcli catalog update --name postgres --user admin --owner Review Comment: This seems odd; can we just use "set" command? considering already use "set" for tag, property, owner is just another type. e.g, "gcli catalog set --name postgres --owner admin --owner-type user". Another possible case is using different commands like "set-tag", "set-owner", with an extensible properties. for example: "gcli catalog set-owner --name postgres --properties owner=admin;type=user". Please think about it. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
