sunxiaojian commented on code in PR #5641: URL: https://github.com/apache/gravitino/pull/5641#discussion_r1853683208
########## docs/cli.md: ########## @@ -468,22 +469,22 @@ gcli tag details --tag tagA gcli tag list ``` -#### Delete a tag +#### Delete tags ```bash -gcli tag delete --tag tagA +gcli tag delete --tag tagA tagB ``` -#### Add a tag to an entity +#### Add tags to an entity ```bash -gcli tag set --name catalog_postgres.hr --tag tagA +gcli tag set --name catalog_postgres.hr --tag tagA tagB Review Comment: > This is confusing too. > > The `--tag` option for the `tag`/`tags` command is weird. > > Or a different approach, where we don't separate tags with spaces in command line. > > Or yet another valid syntax, where the `--tag` option can be specified multiple times. `tag` is command entity, `—tag` is a parameter -- 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: commits-unsubscr...@gravitino.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
