[jira] [Commented] (CALCITE-2744) RelDecorrelator use wrong output map for LogicalAggregate decorrelate

2019-06-10 Thread Danny Chan (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-2744?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16859828#comment-16859828 ] Danny Chan commented on CALCITE-2744: - Thanks [~julianhyde], i still apply the fix because i think

[jira] [Commented] (CALCITE-2744) RelDecorrelator use wrong output map for LogicalAggregate decorrelate

2019-06-09 Thread Danny Chan (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-2744?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16859670#comment-16859670 ] Danny Chan commented on CALCITE-2744: - [~julianhyde] I fired  a fix in 

[jira] [Commented] (CALCITE-2744) RelDecorrelator use wrong output map for LogicalAggregate decorrelate

2019-06-09 Thread Julian Hyde (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-2744?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16859624#comment-16859624 ] Julian Hyde commented on CALCITE-2744: -- While you're in that code, modify the javadoc of MY_AVG

[jira] [Commented] (CALCITE-2744) RelDecorrelator use wrong output map for LogicalAggregate decorrelate

2019-06-09 Thread Julian Hyde (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-2744?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16859621#comment-16859621 ] Julian Hyde commented on CALCITE-2744: -- * It's usually not a good idea if the commit message

[jira] [Commented] (CALCITE-2744) RelDecorrelator use wrong output map for LogicalAggregate decorrelate

2019-06-05 Thread Danny Chan (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-2744?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16857220#comment-16857220 ] Danny Chan commented on CALCITE-2744: - Fixed in 

[jira] [Commented] (CALCITE-2744) RelDecorrelator use wrong output map for LogicalAggregate decorrelate

2019-06-04 Thread Danny Chan (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-2744?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16856331#comment-16856331 ] Danny Chan commented on CALCITE-2744: - [~julianhyde] I think there is some problem with the

[jira] [Commented] (CALCITE-2744) RelDecorrelator use wrong output map for LogicalAggregate decorrelate

2019-06-04 Thread Julian Hyde (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-2744?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16856316#comment-16856316 ] Julian Hyde commented on CALCITE-2744: -- Why two PRs? I reviewed one of them. > RelDecorrelator

[jira] [Commented] (CALCITE-2744) RelDecorrelator use wrong output map for LogicalAggregate decorrelate

2019-06-04 Thread Danny Chan (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-2744?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16856237#comment-16856237 ] Danny Chan commented on CALCITE-2744: - I digged into the code and found that it is not because the