[
https://issues.apache.org/jira/browse/FLINK-3192?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15081373#comment-15081373
]
ASF GitHub Bot commented on FLINK-3192:
---------------------------------------
Github user fhueske commented on the pull request:
https://github.com/apache/flink/pull/1477#issuecomment-168736055
Hi @gallenvara, thanks for this PR!
Explain is a very nice feature for the Table API (and later on for the SQL
interface).
IMO, it would be better to reuse the existing JSON plan and generate the
HIVE visualization locally inside of the Table API. That would avoid to
replicate quite a bit of functionality just for the sake of a different
visualization. Would that work as well?
In general, it is a good idea to discuss changes that modify user-facing
APIs and touch several components/modules before starting to code as pointed
out in our [contribution
guidelines](http://flink.apache.org/contribute-code.html).
> Add explain support to print ast and sql physical execution plan.
> ------------------------------------------------------------------
>
> Key: FLINK-3192
> URL: https://issues.apache.org/jira/browse/FLINK-3192
> Project: Flink
> Issue Type: New Feature
> Components: Table API
> Reporter: GaoLun
> Assignee: GaoLun
> Priority: Minor
> Labels: features
>
> Table API doesn't support sql-explanation now. Add the explain support to
> print ast (abstract syntax tree) and the physical execution plan of sql.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
