This is an automated email from the ASF dual-hosted git repository.

nicholasjiang pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/celeborn.git


The following commit(s) were added to refs/heads/main by this push:
     new df0def670 [CELEBORN-2082] Add the log of excluded workers with high 
workloads
df0def670 is described below

commit df0def6701bb154abc5a1c8967b3425f921110e2
Author: sychen <[email protected]>
AuthorDate: Fri Jul 25 20:56:18 2025 +0800

    [CELEBORN-2082] Add the log of excluded workers with high workloads
    
    ### What changes were proposed in this pull request?
    
    ### Why are the changes needed?
    When workers with higher workloads are excluded, the master does not have a 
clear log.
    
    ### Does this PR introduce _any_ user-facing change?
    No
    
    ### How was this patch tested?
    GA
    
    Closes #3391 from cxzl25/CELEBORN-2082.
    
    Authored-by: sychen <[email protected]>
    Signed-off-by: SteNicholas <[email protected]>
---
 .../service/deploy/master/clustermeta/AbstractMetaManager.java       | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git 
a/master/src/main/java/org/apache/celeborn/service/deploy/master/clustermeta/AbstractMetaManager.java
 
b/master/src/main/java/org/apache/celeborn/service/deploy/master/clustermeta/AbstractMetaManager.java
index 6bee54125..a56b8beb0 100644
--- 
a/master/src/main/java/org/apache/celeborn/service/deploy/master/clustermeta/AbstractMetaManager.java
+++ 
b/master/src/main/java/org/apache/celeborn/service/deploy/master/clustermeta/AbstractMetaManager.java
@@ -294,7 +294,10 @@ public abstract class AbstractMetaManager implements 
IMetadataHandler {
                 && !conf.hasOssStorage())
             || highWorkload)) {
       LOG.warn(
-          "Worker {} (unhealthy disks num: {}) adds to excluded workers", 
worker, unhealthyDiskNum);
+          "Worker {} (unhealthy disks num: {}, high workload: {}) adds to 
excluded workers",
+          worker,
+          unhealthyDiskNum,
+          highWorkload);
       excludedWorkers.add(worker);
     } else if ((availableSlots.get() > 0
             || conf.hasHDFSStorage()

Reply via email to