From 6311090c3c6d3fbece849d0afe6e55045693c198 Mon Sep 17 00:00:00 2001
From: Nathaniel Bauernfeind <natebauernfeind@deephaven.io>
Date: Tue, 5 Mar 2024 13:16:29 -0700
Subject: [PATCH] Test fix 2

---
 .../io/deephaven/engine/context/QueryCompiler.java    |  1 +
 .../deephaven/engine/context/TestQueryCompiler.java   | 11 +----------
 2 files changed, 2 insertions(+), 10 deletions(-)

diff --git a/engine/context/src/main/java/io/deephaven/engine/context/QueryCompiler.java b/engine/context/src/main/java/io/deephaven/engine/context/QueryCompiler.java
index 1e3e0e6bfc7..e8ec7f3facc 100644
--- a/engine/context/src/main/java/io/deephaven/engine/context/QueryCompiler.java
+++ b/engine/context/src/main/java/io/deephaven/engine/context/QueryCompiler.java
@@ -975,6 +975,7 @@ private boolean maybeCreateClassHelper2(
                         // This source actually succeeded in compiling, but was not written because some other source
                         // failed to compile. Let's schedule this work to try again.
                         toRetry.add(request);
+                        return;
                     }
                 }
 
diff --git a/engine/context/src/test/java/io/deephaven/engine/context/TestQueryCompiler.java b/engine/context/src/test/java/io/deephaven/engine/context/TestQueryCompiler.java
index 65d08b6f7f1..a42676f237f 100644
--- a/engine/context/src/test/java/io/deephaven/engine/context/TestQueryCompiler.java
+++ b/engine/context/src/test/java/io/deephaven/engine/context/TestQueryCompiler.java
@@ -299,24 +299,15 @@ public void testMultiCompileWithFailure() throws ExecutionException, Interrupted
                         CompletionStageFutureImpl.make(),
                 };
 
-        Exception firstErr;
         try {
             ExecutionContext.getContext().getQueryCompiler().compile(requests, resolvers);
             // noinspection DataFlowIssue
             throw Assert.statementNeverExecuted();
-        } catch (Exception err) {
-            firstErr = err;
+        } catch (Exception ignored) {
         }
 
         Assert.eqTrue(resolvers[0].getFuture().isDone(), "resolvers[0].getFuture().isDone()");
         Assert.eqTrue(resolvers[1].getFuture().isDone(), "resolvers[0].getFuture().isDone()");
         Assert.neqNull(resolvers[1].getFuture().get(), "resolvers[1].getFuture().get()");
-        try {
-            resolvers[0].getFuture().get();
-            // noinspection DataFlowIssue
-            throw Assert.statementNeverExecuted();
-        } catch (ExecutionException err) {
-            Assert.eq(firstErr, "firstErr", err.getCause(), "err");
-        }
     }
 }