This is an automated email from the ASF dual-hosted git repository.

shaofengshi pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/gravitino.git


The following commit(s) were added to refs/heads/main by this push:
     new fd3a530ae [Minor] Fix list column output in Gravitino CLI (#5871)
fd3a530ae is described below

commit fd3a530aedd328471f35030cfaaf7acfc0a9a327
Author: Justin Mclean <[email protected]>
AuthorDate: Tue Dec 17 16:37:48 2024 +1100

    [Minor] Fix list column output in Gravitino CLI (#5871)
    
    ### What changes were proposed in this pull request?
    
    Stop an error line being output in list columns CLI command.
    
    ### Why are the changes needed?
    
    It should not display an error.
    
    Fix: # N/A
    
    ### Does this PR introduce _any_ user-facing change?
    
    No
    
    ### How was this patch tested?
    
    Tested locally.
---
 .../main/java/org/apache/gravitino/cli/GravitinoCommandLine.java  | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git 
a/clients/cli/src/main/java/org/apache/gravitino/cli/GravitinoCommandLine.java 
b/clients/cli/src/main/java/org/apache/gravitino/cli/GravitinoCommandLine.java
index ab22a5d96..2eaf12585 100644
--- 
a/clients/cli/src/main/java/org/apache/gravitino/cli/GravitinoCommandLine.java
+++ 
b/clients/cli/src/main/java/org/apache/gravitino/cli/GravitinoCommandLine.java
@@ -562,13 +562,17 @@ public class GravitinoCommandLine extends 
TestableCommandLine {
     String catalog = name.getCatalogName();
     String schema = name.getSchemaName();
     String table = name.getTableName();
-    String column = name.getColumnName();
 
     Command.setAuthenticationMode(auth, userName);
 
     if (CommandActions.LIST.equals(command)) {
       newListColumns(url, ignore, metalake, catalog, schema, table).handle();
-    } else if (CommandActions.CREATE.equals(command)) {
+      return;
+    }
+
+    String column = name.getColumnName();
+
+    if (CommandActions.CREATE.equals(command)) {
       String datatype = line.getOptionValue(GravitinoOptions.DATATYPE);
       String comment = line.getOptionValue(GravitinoOptions.COMMENT);
       String position = line.getOptionValue(GravitinoOptions.POSITION);

Reply via email to