mridulm commented on PR #44021:
URL: https://github.com/apache/spark/pull/44021#issuecomment-1861974153

   A couple of comments:
   
   a) IIRC async profiler does not have a way to associate the java thread with 
the underlying linux thread - and so from a future evolution point of view, we 
cannot corelate this with a task's stack traces.
   Given work being done as part of SPARK-45151, is there a way to make this 
work with whatever @yaooqinn added support for ?
   
   b) This PR is creating one file per executor on hdfs - which would end up 
being very expensive as number of executors per application and number of 
applications increase.
   How are we looking to minimize this cost ?
   


-- 
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]


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

Reply via email to