cloud-fan commented on PR #44321:
URL: https://github.com/apache/spark/pull/44321#issuecomment-1858720574

   @mridulm `LiveTask` gets the `TaskInfo` via the listener, this PR sends the 
original `TaskInfo` instance to `DAGScheduler` and thus to the event bus. We 
keep the cloned `TaskInfo` with empty accumulables in `TaskSetManager`, 
assuming the listener won't hold the original `TaskInfo` instance for a long 
time. Built-in listeners are fine, they just aggregate and throw away. User 
listeners may still cause memory issues, but this is out of our control.


-- 
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: reviews-unsubscr...@spark.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to