Skip to content

Commit 90742a4

Browse files
BryanCutlertdcmeehan
authored andcommitted
Fix flaky test TestQueues.testEagerPlanValidation
1 parent 9f54207 commit 90742a4

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

presto-tests/src/test/java/com/facebook/presto/execution/TestQueues.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -401,11 +401,12 @@ public void testEagerPlanValidation()
401401
QueryId secondQuery = createQuery(queryRunner, secondSession, LONG_LASTING_QUERY);
402402
waitForQueryState(queryRunner, secondQuery, QUEUED);
403403

404+
// Force failure during plan validation after queuing has begun
405+
triggerValidationFailure.set(true);
406+
404407
Session thirdSession = builder.setQueryId(QueryId.valueOf("20240930_203743_00003_33333")).build();
405408
QueryId thirdQuery = createQuery(queryRunner, thirdSession, LONG_LASTING_QUERY);
406409

407-
// Force failure during plan validation after queuing has begun
408-
triggerValidationFailure.set(true);
409410
waitForQueryState(queryRunner, thirdQuery, FAILED);
410411

411412
DispatchManager dispatchManager = queryRunner.getCoordinator().getDispatchManager();

0 commit comments

Comments
 (0)