Focus on the ones that matter

henningandersen · May 30, 2024 · 0c66e53 · 0c66e53
1 parent cdc0ea5
commit 0c66e53
Showing 1 changed file with 4 additions and 5 deletions.
diff --git a/server/src/main/java/org/elasticsearch/client/internal/support/AbstractClient.java b/server/src/main/java/org/elasticsearch/client/internal/support/AbstractClient.java
@@ -121,9 +121,9 @@ public final <Request extends ActionRequest, Response extends ActionResponse> Ac
         ActionType<Response> action,
         Request request
     ) {
-        if (EsExecutors.executorName(Thread.currentThread()) != null) {
-            throw new IllegalStateException("can only call synchronous execute on a test thread " + Thread.currentThread());
-        }
+//        if (EsExecutors.executorName(Thread.currentThread()) != null) {
+//            throw new IllegalStateException("can only call synchronous execute on a test thread " + Thread.currentThread());
+//        }
         // assert EsExecutors.executorName(Thread.currentThread()) == null
         // : "can only call synchronous execute on a test thread " + Thread.currentThread();
         PlainActionFuture<Response> actionFuture = new RefCountedFuture<>();
@@ -419,10 +419,9 @@ protected <Request extends ActionRequest, Response extends ActionResponse> void
      */
     // todo: the use of UnsafePlainActionFuture here is quite broad, we should find a better way to be more specific
     // (unless making all usages safe is easy).
-    private static class RefCountedFuture<R extends RefCounted> extends UnsafePlainActionFuture<R> {
+    private static class RefCountedFuture<R extends RefCounted> extends PlainActionFuture<R> {
 
         private RefCountedFuture() {
-            super(ThreadPool.Names.GENERIC);
         }
 
         @Override