jerryshao commented on code in PR #5525: URL: https://github.com/apache/gravitino/pull/5525#discussion_r1836037304
########## docs/cli.md: ########## @@ -352,7 +352,11 @@ gcli column list --metalake metalake_demo --name catalog_postgres.hr.departments ```bash gcli table details --metalake metalake_demo --name catalog_postgres.hr.departments --audit ``` +#### Show tables distribution information +```bash +gcli table details --metalake metalake_demo --name catalog_postgres.hr.departments --distribution +``` Review Comment: I'm curious why do we need a new `--distribution` argument to display the distribution of the table, also for `audit`? Intuitively, `details` should show everything including audit, distribution, and others. IMO, it is not necessary to add arguments one by one to support displaying different attributes. This will potentially lead to too many arguments. -- 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]
