alamb commented on issue #14914: URL: https://github.com/apache/datafusion/issues/14914#issuecomment-2700821082
> For the setting side, how about using the existing `EXPLAIN [ANALYZE] FORMAT <format>` grammar? `sqlparser` can parse and generate a [`AnalyzeFormat`](https://docs.rs/sqlparser/0.54.0/sqlparser/ast/enum.AnalyzeFormat.html), which the planner can use. But our current implementation just drops this format field: > > [datafusion/datafusion/sql/src/statement.rs](https://github.com/apache/datafusion/blob/ed517efcf329de55e7a9d881d6d76d885bc3598d/datafusion/sql/src/statement.rs#L213-L222) > > Lines 213 to 222 in [ed517ef](/apache/datafusion/commit/ed517efcf329de55e7a9d881d6d76d885bc3598d) > > Statement::Explain { > verbose, > statement, > analyze, > format: _, > describe_alias: _, > .. > } => { > self.explain_to_plan(verbose, analyze, DFStatement::Statement(statement)) > } > We can also try to support GRAPHVIZ format in this way, it should be easier to be rendered to svg. This is a great idea @waynexia It seems like AnalyzeFormat currently only supports Text, JSON, and GRAPHVIZ 🤔 https://docs.rs/sqlparser/latest/sqlparser/ast/enum.AnalyzeFormat.html It would be a good way to show graphviz plans -- 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
