Skip to content

Commit

Permalink
Adjust the default value of ackMessageThreadPoolNums to 16 to prevent…
Browse files Browse the repository at this point in the history
… performance bottlenecks during high traffic. (apache#8337)
  • Loading branch information
RongtongJin authored Jul 2, 2024
1 parent c6d3f26 commit 3aa5d19
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,7 @@ public class BrokerConfig extends BrokerIdentity {
private int putMessageFutureThreadPoolNums = Math.min(PROCESSOR_NUMBER, 4);
private int pullMessageThreadPoolNums = 16 + PROCESSOR_NUMBER * 2;
private int litePullMessageThreadPoolNums = 16 + PROCESSOR_NUMBER * 2;
private int ackMessageThreadPoolNums = 3;
private int ackMessageThreadPoolNums = 16;
private int processReplyMessageThreadPoolNums = 16 + PROCESSOR_NUMBER * 2;
private int queryMessageThreadPoolNums = 8 + PROCESSOR_NUMBER;

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -816,7 +816,7 @@ private boolean putMessagePositionInfo(final long offset, final int size, final
long currentLogicOffset = mappedFile.getWrotePosition() + mappedFile.getFileFromOffset();

if (expectLogicOffset < currentLogicOffset) {
log.warn("Build consume queue repeatedly, expectLogicOffset: {} currentLogicOffset: {} Topic: {} QID: {} Diff: {}",
log.warn("Build consume queue repeatedly, expectLogicOffset: {} currentLogicOffset: {} Topic: {} QID: {} Diff: {}",
expectLogicOffset, currentLogicOffset, this.topic, this.queueId, expectLogicOffset - currentLogicOffset);
return true;
}
Expand Down

0 comments on commit 3aa5d19

Please sign in to comment.