s0nskar commented on PR #2700:
URL: https://github.com/apache/celeborn/pull/2700#issuecomment-2303798387

   I think this change can make retrospective debugging hard as we will no 
insight into which workers the shuffle data was written to, what was the 
behaviour of workers during that time. 
   
   Instead of making it completely a debug log IMO we should at least output 
some basic info about the worker host selected for shuffle this will reduce the 
noise but also leave some space for debugging issues.
   
   Ex –
   ```
   Host: 10.152.63.41
   WorkerStatus: WorkerStatus{state=Normal, stateStartTime=1724244049248}
   NumOfSlotsAllocated: 100
   ```


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to