westonpace opened a new issue, #33737:
URL: https://github.com/apache/arrow/issues/33737

   ### Describe the enhancement requested
   
   The old tracing model starts a span when a node starts and ends the span 
when the node marks itself finished.  Some nodes start an additional 
InputReceived span with the above mentioned span as parent.  This makes it 
rather difficult to tell where time is actually being spent because large 
blocks of the span represent idle time.  It does not accurately reflect time 
spent.
   
   I've changed the model to use async scheduler tasks as spans.  In practice, 
this means that there is now a span per fragment.  It may have child spans for 
each of the nodes that runs on the fragment (simple nodes may just mark their 
execution as an event).  This also will allow us to get rid of the 
ExecNode::finsihed_ future as they are no longer really necessary (they 
currently still show up as "waiting for finish" spans that don't really provide 
any useful information).
   
   ### Component(s)
   
   C++


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