joosthooz opened a new pull request #12609:
URL: https://github.com/apache/arrow/pull/12609


   Continuing https://github.com/apache/arrow/pull/12328 and 
https://github.com/apache/arrow/pull/11964.
   The tracing spans were not propagated through all the asynchronous 
constructs, causing some spans to become disconnected from the trace. This PR 
aims to address this.
   Some things left to do:
   - [ ] Possibly add some attributes to the `read_column` span
   - [ ] fix parent/sibling relationships (some of the new spans should 
probably become a child)
   - [ ] Do something about all the `#ifdefs`
   - [ ] Maybe wrap around a `Future`
   - [ ] Check if tracing now works properly for all of the file types, not 
just parquet
   - [ ] @lidavidm mentioned some memory leaks that should be investigated


-- 
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: github-unsubscr...@arrow.apache.org

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


Reply via email to