Re: [PR] [#6493] feat(CLI): Support table format output for Schema and Table command [gravitino]

[via GitHub]

Abyss-lord commented on code in PR #6495:
URL: https://github.com/apache/gravitino/pull/6495#discussion_r1966959635


##########
clients/cli/src/main/java/org/apache/gravitino/cli/outputs/TableFormat.java:
##########
@@ -75,6 +77,14 @@ public static void output(Object entity, CommandContext 
context) {
       new CatalogTableFormat(context).output((Catalog) entity);
     } else if (entity instanceof Catalog[]) {
       new CatalogListTableFormat(context).output((Catalog[]) entity);
+    } else if (entity instanceof Schema) {
+      new SchemaTableFormat(context).output((Schema) entity);
+    } else if (entity instanceof Schema[]) {
+      new SchemaListTableFormat(context).output((Schema[]) entity);
+    } else if (entity instanceof Table) {
+      new TableDetailsTableFormat(context).output((Table) entity);
+    } else if (entity instanceof Table[]) {
+      new TableListTableFormat(context).output((Table[]) entity);

Review Comment:
   
![image](https://github.com/user-attachments/assets/76e2e329-677e-4d3a-8a39-a7e7c19c9cda)
   CICD includes JDK8, and compatibility issues need to be considered.
   



-- 
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