[ 
https://issues.apache.org/jira/browse/FLINK-3192?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15089155#comment-15089155
 ] 

ASF GitHub Bot commented on FLINK-3192:
---------------------------------------

Github user fhueske commented on the pull request:

    https://github.com/apache/flink/pull/1477#issuecomment-169987537
  
    I added two small comments in-line. 
    But there is another thing that came to my mind. A `Table` can be 
constructed from any type of `DataSet` not just a `DataSource`. So, it is 
possible that the input of a query is a partial DataSet program. In that case, 
the Explain output would include the DataSet program as well as the Table 
query. I am not sure if that is actually desirable. Do you think it is possible 
to limit the Explain output to the current query only? Or would you be in favor 
of showing the whole plan?


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

Reply via email to