shaofengshi commented on code in PR #5291: URL: https://github.com/apache/gravitino/pull/5291#discussion_r1835923690
########## docs/cli.md: ########## @@ -169,75 +177,75 @@ gcli metalake list #### Show a metalake details ```bash -gcli metalake details --metalake metalake_demo +gcli metalake details ``` #### Show a metalake audit information ```bash -gcli metalake details --metalake metalake_demo --audit +gcli metalake details --audit ``` #### Create a metalake ```bash -gcli metalake create --metalake my_metalake --comment "This is my metalake" +gcli metalake create --comment "This is my metalake" Review Comment: I know it doesn't have to be, but as a good example (and in a real case), user would specify it inline here, instead of directly configured in the configure file or environment variable. Otherwise, this sample looks strange. -- 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]
