[apache#5932] improve(CLI): Fix the CLI delete the anonymous user

Add error message to the CLI when delete the anonymous user.
Abyss-lord · Feb 10, 2025 · a3f8a32 · a3f8a32
1 parent a84fd18
commit a3f8a32
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/clients/cli/src/main/java/org/apache/gravitino/cli/ErrorMessages.java b/clients/cli/src/main/java/org/apache/gravitino/cli/ErrorMessages.java
@@ -23,6 +23,8 @@
 public class ErrorMessages {
   public static final String CATALOG_EXISTS = "Catalog already exists.";
   public static final String COLUMN_EXISTS = "Column already exists.";
+  public static final String DELETE_ANONYMOUS_USER =
+      "Can't delete anonymous user. This will cause unexpected " + "behavior.";
   public static final String FILESET_EXISTS = "Fileset already exists.";
   public static final String GROUP_EXISTS = "Group already exists.";
   public static final String METALAKE_EXISTS = "Metalake already exists.";

diff --git a/clients/cli/src/main/java/org/apache/gravitino/cli/commands/DeleteUser.java b/clients/cli/src/main/java/org/apache/gravitino/cli/commands/DeleteUser.java
@@ -53,6 +53,10 @@ public DeleteUser(
   public void handle() {
     boolean deleted = false;
 
+    if ("anonymous".equalsIgnoreCase(user)) {
+      exitWithError(ErrorMessages.DELETE_ANONYMOUS_USER);
+    }
+
     if (!AreYouSure.really(force)) {
       return;
     }