This is an automated email from the ASF dual-hosted git repository.
ethanfeng pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/incubator-celeborn.git
The following commit(s) were added to refs/heads/main by this push:
new 59dcd96cb [MINOR] Update log level for
`TransferBufferPool#reserveBuffers` from warn to debug
59dcd96cb is described below
commit 59dcd96cb72bfcefd0f8448943b7a3e8bc7a5f01
Author: SteNicholas <[email protected]>
AuthorDate: Tue Nov 7 11:27:02 2023 +0800
[MINOR] Update log level for `TransferBufferPool#reserveBuffers` from 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 #2076 from SteNicholas/reserve-buffers-log.
Authored-by: SteNicholas <[email protected]>
Signed-off-by: mingji <[email protected]>
---
.../org/apache/celeborn/plugin/flink/buffer/TransferBufferPool.java | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git
a/client-flink/common/src/main/java/org/apache/celeborn/plugin/flink/buffer/TransferBufferPool.java
b/client-flink/common/src/main/java/org/apache/celeborn/plugin/flink/buffer/TransferBufferPool.java
index e1f1ee1f5..cc647a3b6 100644
---
a/client-flink/common/src/main/java/org/apache/celeborn/plugin/flink/buffer/TransferBufferPool.java
+++
b/client-flink/common/src/main/java/org/apache/celeborn/plugin/flink/buffer/TransferBufferPool.java
@@ -106,7 +106,8 @@ public class TransferBufferPool implements BufferRecycler {
listener = creditListener;
}
- logger.warn("reserveBuffers,numCredits: {}, required: {}", numCredits,
numRequiredBuffers);
+ logger.debug(
+ "reserveBuffers: numCredits: {}, requiredBuffers: {}", numCredits,
numRequiredBuffers);
}
if (listener != null) {
listener.notifyAvailableCredits(numCredits);