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

ASF GitHub Bot commented on TAJO-1426:
--------------------------------------

Github user jihoonson commented on the pull request:

    https://github.com/apache/tajo/pull/441#issuecomment-84378037
  
    @navis your work is nice in overall. I left some comments.
    Honestly, I'm writing tests to verify generated plans while working on 
improving join ordering (https://issues.apache.org/jira/browse/TAJO-1352). 
However, your test approach using the explain query looks better than mine. So, 
I'd like to share some challenges which I met.
    * Left and right children of a commutative join (inner and cross) are 
interchangeable with each other. So, the plan can be changed.
    * The order of shuffle keys between execution blocks can be changed.
    
    These challenges should be addressed to validate query plans.


> Support "explain physical" to get physical plan
> -----------------------------------------------
>
>                 Key: TAJO-1426
>                 URL: https://issues.apache.org/jira/browse/TAJO-1426
>             Project: Tajo
>          Issue Type: Improvement
>            Reporter: Navis
>            Assignee: Navis
>            Priority: Minor
>
> Inconvenient to see that in log file.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to