(celeborn) branch branch-0.5 updated: [CELEBORN-1573] Change to debug logging on client side for reserve slots

[ethanfeng]

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

ethanfeng pushed a commit to branch branch-0.5
in repository https://gitbox.apache.org/repos/asf/celeborn.git


The following commit(s) were added to refs/heads/branch-0.5 by this push:
     new 245f01c90 [CELEBORN-1573] Change to debug logging on client side for 
reserve slots
245f01c90 is described below

commit 245f01c90ea55b0b4af60fc8f0ce9cda9c33901f
Author: Aravind Patnam <apat...@linkedin.com>
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 <apat...@linkedin.com>
    Signed-off-by: mingji <fengmingxiao....@alibaba-inc.com>
    (cherry picked from commit 1da2edeb579e1c0c5f2881621abd80bd7d9edd11)
    Signed-off-by: mingji <fengmingxiao....@alibaba-inc.com>
---
 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 82d92d853..566e1bf34 100644
--- a/client/src/main/scala/org/apache/celeborn/client/LifecycleManager.scala
+++ b/client/src/main/scala/org/apache/celeborn/client/LifecycleManager.scala
@@ -651,7 +651,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))