Enable not same executor assertion (elastic#109088)

Enable the assertion introduced in elastic#108934
craigtaverner · Jun 11, 2024 · 72f325b · 72f325b
1 parent fc61326
commit 72f325b
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/server/src/main/java/org/elasticsearch/action/support/PlainActionFuture.java b/server/src/main/java/org/elasticsearch/action/support/PlainActionFuture.java
@@ -407,8 +407,7 @@ public static <T, E extends Exception> T get(CheckedConsumer<PlainActionFuture<T
 
     private boolean assertCompleteAllowed() {
         Thread waiter = sync.getFirstQueuedThread();
-        // todo: reenable assertion once downstream code is updated
-        assert true || waiter == null || allowedExecutors(waiter, Thread.currentThread())
+        assert waiter == null || allowedExecutors(waiter, Thread.currentThread())
             : "cannot complete future on thread "
                 + Thread.currentThread()
                 + " with waiter on thread "