Skip to content

Commit

Permalink
no need to ping same members that are probed
Browse files Browse the repository at this point in the history
Signed-off-by: Matt Wrock <[email protected]>
  • Loading branch information
mwrock committed Nov 6, 2024
1 parent 5dd4375 commit fe1726f
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions components/butterfly/src/server/outbound.rs
Original file line number Diff line number Diff line change
Expand Up @@ -128,13 +128,14 @@ fn run_loop(server: &Server, socket: &UdpSocket, rx_inbound: &AckReceiver, timin
.members_write()
.drain()
.collect::<Vec<_>>();

let mut check_list = server.member_list.check_list_mlr(&server.member_id);
if !members_to_probe.is_empty() {
debug!("Probing {} members in the Probe List.",
members_to_probe.len());

for member in members_to_probe {
trace!("Probing member: {}", member.id);
check_list.retain(|mem| mem.id != member.id);
let addr = member.swim_socket_address();
ping_mlr_smr_rhw(server,
socket,
Expand All @@ -154,8 +155,6 @@ fn run_loop(server: &Server, socket: &UdpSocket, rx_inbound: &AckReceiver, timin
debug!("Zero members in probe_list");
}

let check_list = server.member_list.check_list_mlr(&server.member_id);

let probe_iteration_start = Instant::now();
for member in check_list {
if server.member_list.pingable_mlr(&member) {
Expand Down

0 comments on commit fe1726f

Please sign in to comment.