Skip to content

Commit

Permalink
Should not update Known nodes when refreshed topology is empty
Browse files Browse the repository at this point in the history
Closes redis#3017
  • Loading branch information
jiajunsu committed Oct 18, 2024
1 parent d255b1a commit 43a70f2
Showing 1 changed file with 2 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -120,7 +120,8 @@ private Runnable getTopologyRefreshRunnable(MasterReplicaTopologyRefresh refresh
EventRecorder.getInstance().record(new MasterReplicaTopologyChangedEvent(redisURI, nodes));

if (nodes.isEmpty()) {
LOG.warn("Topology refresh returned no nodes from {}", redisURI);
LOG.warn("Topology refresh returned no nodes from {}, skip refresh Known nodes", redisURI);
return;
}

LOG.debug("New topology: {}", nodes);
Expand Down

0 comments on commit 43a70f2

Please sign in to comment.