Skip to content

Commit

Permalink
[MINOR] Update log level for TransferBufferPool#reserveBuffers from…
Browse files Browse the repository at this point in the history
… warn to debug

### What changes were proposed in this pull request?

Update log level for `TransferBufferPool#reserveBuffers` from warn to debug.

### Why are the changes needed?

The log level of `TransferBufferPool#reserveBuffers` should not be warn, which is normal credit-based shuffle read log.

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

No.

Closes apache#2076 from SteNicholas/reserve-buffers-log.

Authored-by: SteNicholas <programgeek@163.com>
Signed-off-by: mingji <fengmingxiao.fmx@alibaba-inc.com>
  • Loading branch information
SteNicholas authored and FMX committed Nov 7, 2023
1 parent efa22a4 commit 59dcd96
Showing 1 changed file with 2 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -106,7 +106,8 @@ public void reserveBuffers(CreditListener creditListener, int numRequiredBuffers
listener = creditListener;
}

logger.warn("reserveBuffers,numCredits: {}, required: {}", numCredits, numRequiredBuffers);
logger.debug(
"reserveBuffers: numCredits: {}, requiredBuffers: {}", numCredits, numRequiredBuffers);
}
if (listener != null) {
listener.notifyAvailableCredits(numCredits);
Expand Down

0 comments on commit 59dcd96

Please sign in to comment.