ameyc opened a new pull request, #12186: URL: https://github.com/apache/datafusion/pull/12186
## Which issue does this PR close? Closes #11364 ## Rationale for this change Currently ExecutionPlans dont have an identifier associated with them, making it hard to distinguish between the nodes for usecases such as snapshotting continuous pipelines, displaying node metrics in a UI etc. ## What changes are included in this PR? Changes to - 1. `ExecutionPlanProperties` to add node_id `Option<usize>` 2. `ExecutionPlan` to add `with_node_id()` method to return a copy of the ExecutionPlan with assigned node id. 3. Changes to `SessionState` to add node_id annotation to finalized physical plans. 4. Utils in `physical-plan/src/node_id.rs` to traverse ExecutionPlans and generate deterministic ids for the whole tree. ## Are these changes tested? Added asserts to an existing test in `datafusion-examples/src/planner_api.rs`. ## Are there any user-facing changes? No -- 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...@datafusion.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For additional commands, e-mail: github-h...@datafusion.apache.org
