Skip to content

Commit

Permalink
Optimize sending memory management when the audit-proxy config is null
Browse files Browse the repository at this point in the history
  • Loading branch information
doleyzi committed Mar 6, 2024
1 parent b6dffbb commit 33adc77
Showing 1 changed file with 6 additions and 6 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -83,12 +83,12 @@ public SenderGroup(SenderManager senderManager) {
public SenderResult send(ByteBuf dataBuf) {
LinkedBlockingQueue<SenderChannel> channels = channelGroups.get(mIndex);
SenderChannel channel = null;
boolean dataBufHasReleased = false;
boolean dataBuffReleased = false;
try {
if (channels.size() <= 0) {
LOG.error("channels is empty");
dataBuf.release();
dataBufHasReleased = true;
dataBuffReleased = true;
return new SenderResult("channels is empty", 0, false);
}
boolean isOk = false;
Expand Down Expand Up @@ -135,7 +135,7 @@ public SenderResult send(ByteBuf dataBuf) {
if (channel == null) {
LOG.error("can not get a channel");
dataBuf.release();
dataBufHasReleased = true;
dataBuffReleased = true;
return new SenderResult("can not get a channel", 0, false);
}

Expand All @@ -148,10 +148,10 @@ public SenderResult send(ByteBuf dataBuf) {
}
t = channel.getChannel().writeAndFlush(dataBuf).sync().await();
}
dataBufHasReleased = true;
dataBuffReleased = true;
} else {
dataBuf.release();
dataBufHasReleased = true;
dataBuffReleased = true;
}
return new SenderResult(channel.getAddr().getHostString(), channel.getAddr().getPort(), t.isSuccess());
} catch (Throwable ex) {
Expand All @@ -163,7 +163,7 @@ public SenderResult send(ByteBuf dataBuf) {
channel.release();
channels.offer(channel);
}
if (!dataBufHasReleased && dataBuf != null) {
if (!dataBuffReleased && dataBuf != null) {
dataBuf.release();
}
}
Expand Down

0 comments on commit 33adc77

Please sign in to comment.