[jira] [Commented] (CALCITE-1499) Exclude VolcanoPlanner's "originalRoot" from default planning process

2016-12-15 Thread Julian Hyde (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1499?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15752981#comment-15752981 ] Julian Hyde commented on CALCITE-1499: -- The grand plan is that there are no {{addXx

[jira] [Commented] (CALCITE-1499) Exclude VolcanoPlanner's "originalRoot" from default planning process

2016-12-14 Thread Maryann Xue (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1499?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15749842#comment-15749842 ] Maryann Xue commented on CALCITE-1499: -- Julian, I agree with you with the list of s

[jira] [Commented] (CALCITE-1499) Exclude VolcanoPlanner's "originalRoot" from default planning process

2016-12-14 Thread Julian Hyde (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1499?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15748849#comment-15748849 ] Julian Hyde commented on CALCITE-1499: -- That was a rather long answer, but short an

[jira] [Commented] (CALCITE-1499) Exclude VolcanoPlanner's "originalRoot" from default planning process

2016-12-13 Thread Julian Hyde (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1499?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15746831#comment-15746831 ] Julian Hyde commented on CALCITE-1499: -- I think that there is a hierarchy of state.

[jira] [Commented] (CALCITE-1499) Exclude VolcanoPlanner's "originalRoot" from default planning process

2016-12-13 Thread Maryann Xue (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1499?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15746457#comment-15746457 ] Maryann Xue commented on CALCITE-1499: -- Thank you for the review, [~julianhyde]! I

[jira] [Commented] (CALCITE-1499) Exclude VolcanoPlanner's "originalRoot" from default planning process

2016-12-13 Thread Julian Hyde (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1499?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15745710#comment-15745710 ] Julian Hyde commented on CALCITE-1499: -- The change looks OK, but can you add a test

[jira] [Commented] (CALCITE-1499) Exclude VolcanoPlanner's "originalRoot" from default planning process

2016-12-12 Thread Maryann Xue (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1499?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15744418#comment-15744418 ] Maryann Xue commented on CALCITE-1499: -- Sorry, I forgot to mention you, [~julianhyd

[jira] [Commented] (CALCITE-1499) Exclude VolcanoPlanner's "originalRoot" from default planning process

2016-12-07 Thread Maryann Xue (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1499?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15730211#comment-15730211 ] Maryann Xue commented on CALCITE-1499: -- Could you please kindly review https://gith