justinmclean commented on code in PR #5606:
URL: https://github.com/apache/gravitino/pull/5606#discussion_r1859275498
##########
clients/cli/src/main/java/org/apache/gravitino/cli/commands/Command.java:
##########
@@ -69,4 +77,17 @@ protected GravitinoAdminClient buildAdminClient() {
return GravitinoAdminClient.builder(url).build();
}
}
+
+ /**
+ * Outputs the entity to the console.
+ *
+ * @param entity The entity to output.
+ */
+ protected <T> void output(T entity) {
+ if (outputFormat != null && outputFormat.equals("table")) {
+ TableFormat.output(entity);
+ } else {
Review Comment:
Sure, but imagine a user is scripting this, having a "--output $format"
(where $format could be "plain" or "table") would be helpful, and also, more
formats (e.g. @JSON) might be added in the future. Putting that check in would
mean that the script would be more future proof.
--
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]
