[
https://issues.apache.org/jira/browse/CALCITE-1943?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16144583#comment-16144583
]
Julian Hyde commented on CALCITE-1943:
--
Looks good. I will commit when tests pass. I changed
[
https://issues.apache.org/jira/browse/CALCITE-1943?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16137508#comment-16137508
]
Julian Hyde commented on CALCITE-1943:
--
Makes sense. Please document that rationale in the code and
[
https://issues.apache.org/jira/browse/CALCITE-1943?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16127038#comment-16127038
]
Dian Fu commented on CALCITE-1943:
--
In one word, if we don't transform {{A.net_weight}} to
[
https://issues.apache.org/jira/browse/CALCITE-1943?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16127023#comment-16127023
]
Dian Fu commented on CALCITE-1943:
--
You're right. This is related to how to implement the above
[
https://issues.apache.org/jira/browse/CALCITE-1943?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16127009#comment-16127009
]
Julian Hyde commented on CALCITE-1943:
--
I get the sense that you have had design discussions in
[
https://issues.apache.org/jira/browse/CALCITE-1943?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16127003#comment-16127003
]
Julian Hyde commented on CALCITE-1943:
--
Why is {{PREV("A"."net_weight", 0)}} better than
[
https://issues.apache.org/jira/browse/CALCITE-1943?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16126989#comment-16126989
]
Dian Fu commented on CALCITE-1943:
--
I have created a PR: https://github.com/apache/calcite/pull/516
[
https://issues.apache.org/jira/browse/CALCITE-1943?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16126885#comment-16126885
]
Dian Fu commented on CALCITE-1943:
--
For {{NavigationExpander}}, currently I have not got a quite