diff --git a/clients/cli/src/main/java/org/apache/gravitino/cli/CommandContext.java b/clients/cli/src/main/java/org/apache/gravitino/cli/CommandContext.java index fdf98f57286..fc713ba5775 100644 --- a/clients/cli/src/main/java/org/apache/gravitino/cli/CommandContext.java +++ b/clients/cli/src/main/java/org/apache/gravitino/cli/CommandContext.java @@ -25,7 +25,6 @@ /* Context for a command */ public class CommandContext { - public static final String DEFAULT_URL = "http://localhost:8090"; private final boolean force; private final boolean ignoreVersions; private final String outputFormat; @@ -127,7 +126,7 @@ private String getUrl() { } // Return the default localhost URL - return DEFAULT_URL; + return GravitinoCommandLine.DEFAULT_URL; } private boolean getIgnore() { diff --git a/clients/cli/src/main/java/org/apache/gravitino/cli/CommandHandler.java b/clients/cli/src/main/java/org/apache/gravitino/cli/CommandHandler.java index 66090219887..79462e81021 100644 --- a/clients/cli/src/main/java/org/apache/gravitino/cli/CommandHandler.java +++ b/clients/cli/src/main/java/org/apache/gravitino/cli/CommandHandler.java @@ -25,9 +25,6 @@ public abstract class CommandHandler { public static final Joiner COMMA_JOINER = Joiner.on(", ").skipNulls(); - - public static final String DEFAULT_URL = "http://localhost:8090"; - private String authEnv; private boolean authSet = false; diff --git a/clients/cli/src/main/java/org/apache/gravitino/cli/commands/ClientVersion.java b/clients/cli/src/main/java/org/apache/gravitino/cli/commands/ClientVersion.java index 0a466c048f4..28c702e39fb 100644 --- a/clients/cli/src/main/java/org/apache/gravitino/cli/commands/ClientVersion.java +++ b/clients/cli/src/main/java/org/apache/gravitino/cli/commands/ClientVersion.java @@ -44,6 +44,6 @@ public void handle() { } catch (Exception exp) { exitWithError(exp.getMessage()); } - printInformation("Apache Gravitino " + version); + printResults("Apache Gravitino " + version); } } diff --git a/clients/cli/src/main/java/org/apache/gravitino/cli/commands/ServerVersion.java b/clients/cli/src/main/java/org/apache/gravitino/cli/commands/ServerVersion.java index d4f0cac0878..b94076db69f 100644 --- a/clients/cli/src/main/java/org/apache/gravitino/cli/commands/ServerVersion.java +++ b/clients/cli/src/main/java/org/apache/gravitino/cli/commands/ServerVersion.java @@ -45,6 +45,6 @@ public void handle() { exitWithError(exp.getMessage()); } - printInformation("Apache Gravitino " + version); + printResults("Apache Gravitino " + version); } } diff --git a/clients/cli/src/test/java/org/apache/gravitino/cli/TestCommandContext.java b/clients/cli/src/test/java/org/apache/gravitino/cli/TestCommandContext.java index 5a66629b7d6..5bdd822b333 100644 --- a/clients/cli/src/test/java/org/apache/gravitino/cli/TestCommandContext.java +++ b/clients/cli/src/test/java/org/apache/gravitino/cli/TestCommandContext.java @@ -41,7 +41,7 @@ void setUp() { public void testCreateCommandContextWithDefaults() { CommandContext commandContext = new CommandContext(mockCommandLine); - Assertions.assertEquals(CommandContext.DEFAULT_URL, commandContext.url()); + Assertions.assertEquals(GravitinoCommandLine.DEFAULT_URL, commandContext.url()); Assertions.assertFalse(commandContext.ignoreVersions()); Assertions.assertFalse(commandContext.force()); Assertions.assertEquals(Command.OUTPUT_FORMAT_PLAIN, commandContext.outputFormat());