This is an automated email from the ASF dual-hosted git repository.
ethanfeng pushed a commit to branch branch-0.4
in repository https://gitbox.apache.org/repos/asf/celeborn.git
The following commit(s) were added to refs/heads/branch-0.4 by this push:
new eff2ef151 [CELEBORN-1573] Change to debug logging on client side for
reserve slots
eff2ef151 is described below
commit eff2ef151ec3d3d20e668d8be29e118add55dddf
Author: Aravind Patnam <[email protected]>
AuthorDate: Thu Aug 22 14:49:45 2024 +0800
[CELEBORN-1573] Change to debug logging on client side for reserve slots
### What changes were proposed in this pull request?
Change client side reserve slots log to debug.
### Why are the changes needed?
in a large fleet of workers such as ours, it is noisy for the user in the
driver logs.
### Does this PR introduce _any_ user-facing change?
No
### How was this patch tested?
N/A
Closes #2700 from akpatnam25/CELEBORN-1573.
Authored-by: Aravind Patnam <[email protected]>
Signed-off-by: mingji <[email protected]>
(cherry picked from commit 1da2edeb579e1c0c5f2881621abd80bd7d9edd11)
Signed-off-by: mingji <[email protected]>
---
client/src/main/scala/org/apache/celeborn/client/LifecycleManager.scala | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
a/client/src/main/scala/org/apache/celeborn/client/LifecycleManager.scala
b/client/src/main/scala/org/apache/celeborn/client/LifecycleManager.scala
index e73106d21..88bbcab79 100644
--- a/client/src/main/scala/org/apache/celeborn/client/LifecycleManager.scala
+++ b/client/src/main/scala/org/apache/celeborn/client/LifecycleManager.scala
@@ -563,7 +563,7 @@ class LifecycleManager(val appUniqueId: String, val conf:
CelebornConf) extends
replyRegisterShuffle(RegisterShuffleResponse(StatusCode.SLOT_NOT_AVAILABLE,
Array.empty))
return
case StatusCode.SUCCESS =>
- logInfo(s"OfferSlots for $shuffleId Success!Slots Info:
${res.workerResource}")
+ logDebug(s"OfferSlots for $shuffleId Success!Slots Info:
${res.workerResource}")
case StatusCode.WORKER_EXCLUDED =>
logInfo(s"OfferSlots for $shuffleId failed due to all workers be
excluded!")
replyRegisterShuffle(RegisterShuffleResponse(StatusCode.WORKER_EXCLUDED,
Array.empty))
