Implemented active checks, reverted test case, minor javadoc nit

- Added ACTIVE checks to all routes in AdminSparkServer using VeniceParentControllerRegionStateHandler wrapper class - Reverted CreateVersionTest to previous commit without ParentControllerRegionState.ACTIVE - Removed some minor parentheses in ParentControllerRegionState
linkedin · Jul 29, 2024 · 8e492e1 · 8e492e1
1 parent 20bd2e3
commit 8e492e1
Show file tree

Hide file tree

Showing 3 changed files with 310 additions and 118 deletions.
diff --git a/...-controller/src/main/java/com/linkedin/venice/controller/ParentControllerRegionState.java b/...-controller/src/main/java/com/linkedin/venice/controller/ParentControllerRegionState.java
@@ -2,8 +2,8 @@
 
 /**
  * Enum representing the state of the region where the parent controller resides.
- * (i.e., Region dc-0 is ACTIVE while Region dc-1 is PASSIVE
- * This means that ParentController in dc-0 is serving requests while ParentController in dc-1 is rejecting requests)
+ * i.e., Region dc-0 is ACTIVE while Region dc-1 is PASSIVE
+ * This means that ParentController in dc-0 is serving requests while ParentController in dc-1 is rejecting requests
  */
 public enum ParentControllerRegionState {
   /** The region is active, so the parent controller in the region is serving requests */