justinmclean commented on code in PR #5792: URL: https://github.com/apache/gravitino/pull/5792#discussion_r1875354352
########## docs/cli.md: ########## @@ -822,6 +822,12 @@ null, boolean, byte, ubyte, short, ushort, integer, uinteger, long, ulong, float In addition decimal(precision,scale) and varchar(length). +#### Show a column's audit information + +```bash +gcli column details --name catalog_postgres.hr.departments.name --audit Review Comment: It could be done that way. However, having fewer commands and using flags where possible is better from a good practice point of view. (see https://clig.dev/#arguments-and-flags) If you rearrange the command this way, it probably makes a bit more sense: `cli column details --audit --name catalog_postgres.hr.departments.name` In that, there are different details about an entity that can be displayed, audit information being just one, see the table command for an example with multiple display options. -- 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]
