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