[GitHub] [arrow] westonpace opened a new issue, #33737: [C++] Simplify tracing in exec plan

[GitBox <gi...@apache.org>]

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: issues-unsubscribe@arrow.apache.org.apache.org

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

[GitHub] [arrow] westonpace closed issue #33737: [C++] Simplify tracing in exec plan

["westonpace (via GitHub)" <gi...@apache.org>]

westonpace closed issue #33737: [C++] Simplify tracing in exec plan
URL: https://github.com/apache/arrow/issues/33737


-- 
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: issues-unsubscribe@arrow.apache.org

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