Skip to content

Commit

Permalink
[fix][client] Fix memory leak in ClientCnx.newLookup when there's Too…
Browse files Browse the repository at this point in the history
…ManyRequestsException (apache#23971)

(cherry picked from commit 6a61464)
  • Loading branch information
lhotari authored and mukesh-ctds committed Feb 20, 2025
1 parent b8ccae1 commit 10ec13f
Showing 1 changed file with 1 addition and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -925,6 +925,7 @@ public CompletableFuture<LookupDataResult> newLookup(ByteBuf request, long reque
if (maxLookupRequestSemaphore.tryAcquire()) {
waitingLookupRequests.add(Pair.of(requestId, Pair.of(request, future)));
} else {
request.release();
if (log.isDebugEnabled()) {
log.debug("{} Failed to add lookup-request into waiting queue", requestId);
}
Expand Down

0 comments on commit 10ec13f

Please sign in to comment.