apache/celeborn
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

[CELEBORN-1792][FOLLOWUP] Suppress noisy logs when there is no memory pressure

Browse Source 
### What changes were proposed in this pull request?
Remove unnecessary logs.
<img width="921" alt="截屏2025-02-05 14 29 43" src="https://github.com/user-attachments/assets/ab05f96b-abc0-4d68-a416-82655cd7cd13" />

### Why are the changes needed?
Do not need to log the memory state when there is no memory pressure.

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

### How was this patch tested?
GA and cluster tests.

Closes #3084 from FMX/b1792-1.

Authored-by: mingji <fengmingxiao.fmx@alibaba-inc.com>
Signed-off-by: Shuang <lvshuang.xjs@alibaba-inc.com>
This commit is contained in:
mingji 2025-02-08 09:44:08 +08:00 committed by Shuang
parent e78c9b8ab5
commit 2e4f36f9d4
1 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
worker/src/main/java/org/apache/celeborn/service/deploy/worker/memory/MemoryManager.java
View File

@ -330,7 +330,10 @@ public class MemoryManager {
public void switchServingState() {
ServingState lastState = servingState;
servingState = currentServingState();
logger.info("Serving state transformed from {} to {}", lastState, servingState);
if (servingState != lastState) {
logger.info("Serving state transformed from {} to {}", lastState, servingState);
}
switch (servingState) {
case PUSH_PAUSED:
if (canResumeByPinnedMemory()) {