[
https://issues.apache.org/jira/browse/CALCITE-5503?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17692185#comment-17692185
]
Moritz Mack commented on CALCITE-5503:
--
(y) Thanks [~julianhyde] and [~libenchao]
>
[
https://issues.apache.org/jira/browse/CALCITE-5503?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17683251#comment-17683251
]
Moritz Mack commented on CALCITE-5503:
--
Sounds good to me (y)
The PR is ready to merge. Thanks a
[
https://issues.apache.org/jira/browse/CALCITE-5503?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17682904#comment-17682904
]
Moritz Mack edited comment on CALCITE-5503 at 2/1/23 8:39 AM:
--
[
https://issues.apache.org/jira/browse/CALCITE-5503?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17682904#comment-17682904
]
Moritz Mack commented on CALCITE-5503:
--
[~julianhyde] Thanks for the pointer! Yes it's possible to
[
https://issues.apache.org/jira/browse/CALCITE-5503?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Moritz Mack updated CALCITE-5503:
-
Summary: Repeated nodes in a DAG plan are not reused in
CheapestPlanReplacer (was: Memoize
[
https://issues.apache.org/jira/browse/CALCITE-5503?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17681983#comment-17681983
]
Moritz Mack commented on CALCITE-5503:
--
Thanks [~julianhyde] !
I'll rephrase the commit message
Moritz Mack created CALCITE-5503:
Summary: Memoize visited nodes in CheapestPlanReplacer
Key: CALCITE-5503
URL: https://issues.apache.org/jira/browse/CALCITE-5503
Project: Calcite
Issue