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

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


The following commit(s) were added to refs/heads/main by this push:
     new bcdadae248 throws exception when unknown compaction kind seen (#3778)
bcdadae248 is described below

commit bcdadae248bd54e13aae9056bb10e8f6e82ca758
Author: Keith Turner <ktur...@apache.org>
AuthorDate: Mon Sep 25 17:30:18 2023 -0400

    throws exception when unknown compaction kind seen (#3778)
---
 .../compactor/src/main/java/org/apache/accumulo/compactor/ExtCEnv.java | 3 ++-
 .../src/main/java/org/apache/accumulo/tserver/tablet/MajCEnv.java      | 3 ++-
 2 files changed, 4 insertions(+), 2 deletions(-)

diff --git 
a/server/compactor/src/main/java/org/apache/accumulo/compactor/ExtCEnv.java 
b/server/compactor/src/main/java/org/apache/accumulo/compactor/ExtCEnv.java
index e68690bdba..a9b72ae9bc 100644
--- a/server/compactor/src/main/java/org/apache/accumulo/compactor/ExtCEnv.java
+++ b/server/compactor/src/main/java/org/apache/accumulo/compactor/ExtCEnv.java
@@ -106,8 +106,9 @@ public class ExtCEnv implements CompactionEnv {
         return TCompactionReason.CHOP;
       case SELECTOR:
       case SYSTEM:
-      default:
         return TCompactionReason.SYSTEM;
+      default:
+        throw new IllegalStateException("Unknown compaction kind " + 
job.getKind());
     }
   }
 
diff --git 
a/server/tserver/src/main/java/org/apache/accumulo/tserver/tablet/MajCEnv.java 
b/server/tserver/src/main/java/org/apache/accumulo/tserver/tablet/MajCEnv.java
index f878715e14..0ea93196f2 100644
--- 
a/server/tserver/src/main/java/org/apache/accumulo/tserver/tablet/MajCEnv.java
+++ 
b/server/tserver/src/main/java/org/apache/accumulo/tserver/tablet/MajCEnv.java
@@ -89,8 +89,9 @@ public class MajCEnv implements FileCompactor.CompactionEnv {
         return TCompactionReason.CHOP;
       case SELECTOR:
       case SYSTEM:
-      default:
         return TCompactionReason.SYSTEM;
+      default:
+        throw new IllegalStateException("Unknown compaction kind " + kind);
     }
   }
 }

Reply via email to