[ 
https://issues.apache.org/jira/browse/CALCITE-5715?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Benchao Li updated CALCITE-5715:
--------------------------------
    Description: 
We already have many similar usages (prune old nodes) to reduce the search 
scope, I propose to also add this for {{ProjectMergeRule}}. Do you have any 
concerns about this?

The reason I propose to do this optimization for {{ProjectMergeRule}} is that I 
met a problem that two sets will point to each other after projection merging, 
and the planner will run into dead loop. (It's hard to show the details, I 
haven't got a simple reproducible demo yet)

  was:
We already have many similar usages (prune old nodes) to reduce the search 
scope, I propose to also add this for {{ProjectionMergeRule}}. Do you have any 
concerns about this?

The reason I propose to do this optimization for {{ProjectionMergeRule}} is 
that I met a problem that two sets will point to each other after projection 
merging, and the planner will run into dead loop. (It's hard to show the 
details, I haven't got a simple reproducible demo yet)


> Prune old nodes after projection merging in ProjectMergeRule
> ------------------------------------------------------------
>
>                 Key: CALCITE-5715
>                 URL: https://issues.apache.org/jira/browse/CALCITE-5715
>             Project: Calcite
>          Issue Type: Improvement
>          Components: core
>    Affects Versions: 1.34.0
>            Reporter: Benchao Li
>            Priority: Major
>
> We already have many similar usages (prune old nodes) to reduce the search 
> scope, I propose to also add this for {{ProjectMergeRule}}. Do you have any 
> concerns about this?
> The reason I propose to do this optimization for {{ProjectMergeRule}} is that 
> I met a problem that two sets will point to each other after projection 
> merging, and the planner will run into dead loop. (It's hard to show the 
> details, I haven't got a simple reproducible demo yet)



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to