GEODE-9977: Remove defunct RedisCommandSupportLevel.INTERNAL type (ap…

…ache#7283) Authored-by: Donal Evans <[email protected]>
jmelchio · Jan 20, 2022 · b349cb8 · b349cb8
1 parent dccfc18
commit b349cb8
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 10 deletions.
diff --git a/...edis/src/main/java/org/apache/geode/redis/internal/commands/RedisCommandSupportLevel.java b/...edis/src/main/java/org/apache/geode/redis/internal/commands/RedisCommandSupportLevel.java
@@ -19,6 +19,5 @@
 public enum RedisCommandSupportLevel {
   SUPPORTED,
   UNSUPPORTED,
-  UNKNOWN,
-  INTERNAL
+  UNKNOWN
 }
diff --git a/geode-for-redis/src/main/java/org/apache/geode/redis/internal/commands/RedisCommandType.java b/geode-for-redis/src/main/java/org/apache/geode/redis/internal/commands/RedisCommandType.java
@@ -16,7 +16,6 @@
 package org.apache.geode.redis.internal.commands;
 
 import static org.apache.geode.redis.internal.RedisConstants.ERROR_SYNTAX;
-import static org.apache.geode.redis.internal.commands.RedisCommandSupportLevel.INTERNAL;
 import static org.apache.geode.redis.internal.commands.RedisCommandSupportLevel.SUPPORTED;
 import static org.apache.geode.redis.internal.commands.RedisCommandSupportLevel.UNSUPPORTED;
 import static org.apache.geode.redis.internal.commands.RedisCommandType.Flag.ADMIN;
@@ -447,11 +446,6 @@ public boolean isUnsupported() {
     return supportLevel == UNSUPPORTED;
   }
 
-
-  public boolean isInternal() {
-    return supportLevel == INTERNAL;
-  }
-
   public boolean isUnknown() {
     return supportLevel == RedisCommandSupportLevel.UNKNOWN;
   }

diff --git a/...java/org/apache/geode/redis/internal/commands/executor/server/COMMANDCommandExecutor.java b/...java/org/apache/geode/redis/internal/commands/executor/server/COMMANDCommandExecutor.java
@@ -62,8 +62,7 @@ private RedisResponse executeNoSubcommand(ExecutionHandlerContext context) {
     List<Object> response = new ArrayList<>();
 
     for (RedisCommandType type : RedisCommandType.values()) {
-      if (type.isInternal()
-          || type.isUnknown()
+      if (type.isUnknown()
           || (type.isUnsupported() && !context.allowUnsupportedCommands())
           || type == RedisCommandType.QUIT) {
         continue;