justinmclean commented on code in PR #6503:
URL: https://github.com/apache/gravitino/pull/6503#discussion_r1967318589
##########
clients/cli/src/main/java/org/apache/gravitino/cli/outputs/TableFormat.java:
##########
@@ -643,4 +646,31 @@ public String getOutput(Table[] tables) {
return getTableFormat(column);
}
}
+
+ /**
+ * Formats a single {@link Audit} instance into a four-column table display.
Displays audit
+ * details, including creator, create time, modified, and modify time.
+ */
+ static final class AuditTableFormat extends TableFormat<Audit> {
+ public AuditTableFormat(CommandContext context) {
+ super(context);
+ }
+
+ /** {@inheritDoc} */
+ @Override
+ public String getOutput(Audit audit) {
+ Column columnCreator = new Column(context, "creator");
+ Column columnCreateTime = new Column(context, "create time");
+ Column columnModified = new Column(context, "modified");
+ Column columnModifyTime = new Column(context, "modify time");
+
Review Comment:
These headings could be improved - I'd suggest "creator," "creation date,"
"modifier", "modified date." You could use time instead of date, but it's
really a full date and time. or could also use "created by", "created at",
"modified by", "modified at".
--
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
