shaofengshi commented on code in PR #5792: URL: https://github.com/apache/gravitino/pull/5792#discussion_r1881590418
########## 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: > I didn't mean this is a blocker. I was simply thinking that if we want to consolidate all audit kind of operations to one command, under which a user can specify the objects to check via flags. My understanding is that auditing, in general, is a cross-cutting concern, rather than something I want to retrieve from `column details`. Good point; I like this idea; It would be better to put all audit related actions together, so that the guy who checks the auditing logs can quickly check all the logs across different entities. -- 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]
