tustvold opened a new issue #219:
URL: https://github.com/apache/arrow-datafusion/issues/219


   The ability to get a graphviz render of the LogicalPlans is awesome, it 
would be even more awesome to have a similar facility for PhysicalPlans.
   
   The way I'd envisage this working is with a new method added to the 
ExecutionPlan trait
   
   ```
   fn fmt_for_explain(&self, f: &mut fmt::Formatter) -> fmt::Result;
   ```
   
   This is copied wholesale from UserDefinedLogicalNode and would function in 
much the same way.  This would have a default placeholder implementation to 
avoid this being a breaking change.
   
   A public free function would then be added to visit each node in a provided 
ExecutionPlan tree in turn and use this `fmt_for_explain` along with the 
existing schema function to produce a graphviz representation. (This could also 
be a trait function if preferred).
   
   FYI @alamb 


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

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to