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 (apache#21205)
  • Loading branch information
hanmz authored Sep 21, 2023
1 parent e5b0f17 commit 66271e3
Showing 1 changed file with 1 addition and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -634,6 +634,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 66271e3

Please sign in to comment.