Skip to content

Commit

Permalink
Merge pull request #31385 from vespa-engine/lesters/remove-async-asse…
Browse files Browse the repository at this point in the history
…rtion

Remove unnecessary assertion
  • Loading branch information
bratseth authored Jun 1, 2024
2 parents 35f940a + 8c9faa7 commit 22c93a7
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -141,7 +141,7 @@ public void testAsyncRendering() throws InterruptedException, ExecutionException
});
assertFalse(future.isDone());
result = render(new Result(new Query(), newHitGroup(tokenStream, "token_stream")));
assertTrue(future.isDone()); // Renderer waits for async completion
future.join(); // Renderer waits for async completion

} finally {
executor.shutdownNow();
Expand Down

0 comments on commit 22c93a7

Please sign in to comment.