liupc commented on issue #23580: [SPARK-26660]Add warning logs for large 
taskBinary size
URL: https://github.com/apache/spark/pull/23580#issuecomment-455963082
 
 
   @maropu I don't understand what you mean by "another better place to detect 
both cases simultaneously: whole closure size and task size you described?". As 
my point of view, broadcasting large task binary (this broadcast is shared 
among all tasks) and the serialized task in TaskSetManager are two different 
things. I think is'ok to print a warning log in before broadcasting large task 
binary. I also made some log changes in the reading of broadcast to make the 
reading size more clear.
   
   @srowen @jerryshao any suggestions?

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to