shaofengshi commented on code in PR #5783:
URL: https://github.com/apache/gravitino/pull/5783#discussion_r1888008066


##########
docs/cli.md:
##########
@@ -688,10 +694,23 @@ gcli group grant --group groupA --role admin
 ```
 
 #### Remove a role from a group
+
 ```bash
 gcli group revoke --group groupA --role admin
 ```
 
+### Grant a privilege
+
+```bash
+gcli catalog grant --name catalog_postgres --role admin --privilege 
create_table modify_table

Review Comment:
   I checked the Java API, I think the Java API is okay 
(https://github.com/apache/gravitino/blob/main/clients/client-java/src/main/java/org/apache/gravitino/client/GravitinoClient.java#L423).
 The only thing is, you need to construct a MetadataObject which need to know 
the entity type. But even without the type, you can get that easily by invoke 
the Java API to get the catalog object, then knowing it is a catalog, schema, 
table, fileset or topic, is it?
   
   @jerqi @jerryshao your comments are welcomed as well.



-- 
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]

Reply via email to