ameyc opened a new issue, #11364:
URL: https://github.com/apache/datafusion/issues/11364

   ### Is your feature request related to a problem or challenge?
   
   Currently execution plans do not have an id associated with them this makes 
comparison of metrics across the runs. Additionally we would like to add a UI 
to our project to display the metrics as well snapshot our plans.
   
   Looking at the code, it seems to me that the the `physical_planner` [does 
record the node 
index](https://github.com/apache/datafusion/blob/main/datafusion/core/src/physical_planner.rs#L555C17-L555C30)
 which is not [passed down to 
`map_logical_node_to_physical`](https://github.com/apache/datafusion/blob/main/datafusion/core/src/physical_planner.rs#L697-L702).
   
   
   
   ### Describe the solution you'd like
   
   It would quite trivial to pass the node index to ExecutionPlan creation step 
and add additional field `operator_id`  `plan_id` to ExecutionPlanProperties.
   
   ### Describe alternatives you've considered
   
   We looked into using `task_id` field, but it always seems to be set to None.
   
   ### Additional context
   
   _No response_


-- 
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.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

Reply via email to