You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by "alamb (via GitHub)" <gi...@apache.org> on 2023/06/21 17:12:53 UTC

[GitHub] [arrow-datafusion] alamb commented on pull request #6726: feat: Add graphviz display format for execution plan.

alamb commented on PR #6726:
URL: https://github.com/apache/arrow-datafusion/pull/6726#issuecomment-1601263853

   >  @alamb Yes, I have read that before, but I prefer this approach because it's more structural, type safe, and easier to maintain. And I'm planning to modify logical pla n graphviz format later.
   
   That makes sense -- thank you for the explanation. As long as we don't have two different ways of making graphviz I think your approach makes sense
   


-- 
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-unsubscribe@arrow.apache.org

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