[GitHub] [arrow] joosthooz commented on pull request #33738: GH-33737: [C++] simplify exec plan tracing

joosthooz commented on PR #33738: URL: https://github.com/apache/arrow/pull/33738#issuecomment-1400316462 Hi, I gave this branch a spin, and it seems that the nesting has become inconsistent: ![image](https://user-images.githubusercontent.com/1442581/214045948-eb3f077b-da26-457c-ab42-b5

[GitHub] [arrow] joosthooz commented on pull request #33738: GH-33737: [C++] simplify exec plan tracing

joosthooz commented on PR #33738: URL: https://github.com/apache/arrow/pull/33738#issuecomment-1396819003 Nice, I think the WriteAndCheckBackpressure span is important because that's where the backpressure is checked and also it performs some work combining staged batches (in `PopStagedBatc

[GitHub] [arrow] joosthooz commented on pull request #33738: GH-33737: [C++] simplify exec plan tracing

joosthooz commented on PR #33738: URL: https://github.com/apache/arrow/pull/33738#issuecomment-1387311099 In the figure, why does `WaitForFinish(SinkNode:)` end earlier than the `ScalarAggregate`? Can we add a name (and maybe even an id number in case there are multiple) to the names so tha