Skip to content

Commit

Permalink
Fix flaky test for CoordinatorReconfigureNodeMaxTest#testReconfigureN…
Browse files Browse the repository at this point in the history
…odeMax
  • Loading branch information
maobaolong committed Jan 5, 2025
1 parent 3b0521a commit efd894d
Showing 1 changed file with 2 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -131,9 +131,11 @@ public void testReconfigureNodeMax() throws Exception {
assertEquals(5, info.getServerToPartitionRanges().keySet().size());

// case3: recover its value to 10
Thread.sleep(1000L);
try (FileWriter fileWriter = new FileWriter(tempConfFilePath)) {
fileWriter.append(CoordinatorConf.COORDINATOR_SHUFFLE_NODES_MAX.key() + " " + 10);
}
Thread.sleep(1000L);
Awaitility.await().timeout(2, TimeUnit.SECONDS).until(() -> nodeMax.get() == 10);
info = shuffleWriteClient.getShuffleAssignments("app1", 0, 10, 1, TAGS, SERVER_NUM + 10, -1);
assertEquals(10, info.getServerToPartitionRanges().keySet().size());
Expand Down

0 comments on commit efd894d

Please sign in to comment.