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()