westonpace commented on PR #15253: URL: https://github.com/apache/arrow/pull/15253#issuecomment-1374914790
Hmm, it appears that OpenTelemetry was depending on the `finished_` future to mark how long a node took to run. I'm not entirely sure I agree with the concept of span-per-node (I think a span should be tied to something more like a thread task). However, this PR, as it stands, will very much break OT output (spans won't be finished and so simply will not output). I'll try and put up a PR on Monday to fix OT that can come before this one. -- 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]
