WXPNG commented on PR #12755:
URL: https://github.com/apache/seatunnel/pull/12755#issuecomment-6258462458

   Thanks @SEZ9! To close the two remaining points on head def95cccf:
   
   - **F2**: `joinViaPossibleMembers()` resets the master-address-to-join via
     `if (clearedAny && !clusterService.isJoined()) 
clusterService.setMasterAddressToJoin(null);`
     before `continue`, so a stale master is no longer pinned.
   - **F1/F3 behavior**: the retry loop now refreshes an empty candidate list 
each pass
     (`if (possibleAddresses.isEmpty()) possibleAddresses = 
getPossibleAddresses();`), so a
     discovery-based worker re-runs discovery instead of idling, covered by the 
new
     `discoveryWorkerStartedWithEmptyDiscoveryShouldEventuallyJoin` test.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to