Skip to content

Commit

Permalink
Issue 7361 : Fix flakey test "testOwnership"
Browse files Browse the repository at this point in the history
Fixes the race condition to eliminate the flakiness in "testOwnership" unit test.
Signed-off-by: Bhupender Y <[email protected]>
Co-authored-by: Raúl Gracia <[email protected]>
  • Loading branch information
Bhupender-Y authored Jan 29, 2024
1 parent a0758b9 commit 2362d68
Showing 1 changed file with 0 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -169,8 +169,6 @@ public void testOwnership() throws Exception {
assertNotNull(bucketServices);
assertEquals(1, bucketServices.size());

assertTrue(watermarkingService.releaseBucketOwnership(1, dummyProcessId).join());
//For id which is not existing, it will return as true.
assertTrue(watermarkingService.releaseBucketOwnership(1, dummyProcessId).join());
assertTrue(watermarkingService.releaseBucketOwnership(2, dummyProcessId).join());
//All the bucket services get released from dummy process. Now actual owner will occupy this.
Expand Down

0 comments on commit 2362d68

Please sign in to comment.