Skip to content

Commit

Permalink
[CELEBORN-1573] Change to debug logging on client side for reserve slots
Browse files Browse the repository at this point in the history
### 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 apache#2700 from akpatnam25/CELEBORN-1573.

Authored-by: Aravind Patnam <apatnam@linkedin.com>
Signed-off-by: mingji <fengmingxiao.fmx@alibaba-inc.com>
  • Loading branch information
Aravind Patnam authored and FMX committed Aug 22, 2024
1 parent 1f497cb commit 1da2ede
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -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))
Expand Down

0 comments on commit 1da2ede

Please sign in to comment.