Skip to content

Commit

Permalink
[fix][broker] replicator leak when removeReplicator in NonPersistentT…
Browse files Browse the repository at this point in the history
…opic (#21205)
  • Loading branch information
hanmz authored and Technoboy- committed Sep 25, 2023
1 parent 786f892 commit b877344
Showing 1 changed file with 1 addition and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -607,6 +607,7 @@ CompletableFuture<Void> removeReplicator(String remoteCluster) {

replicators.get(remoteCluster).disconnect().thenRun(() -> {
log.info("[{}] Successfully removed replicator {}", name, remoteCluster);
replicators.remove(remoteCluster);

}).exceptionally(e -> {
log.error("[{}] Failed to close replication producer {} {}", topic, name, e.getMessage(), e);
Expand Down

0 comments on commit b877344

Please sign in to comment.