calcite icon indicating copy to clipboard operation
calcite copied to clipboard

[CALCITE-5742] SubstitutionVisitor Causes Infinite Loop when using AggregateOnCalcToAggregateUnifyRule

Open junheleo opened this issue 2 years ago • 5 comments

The input of Calc Under Aggregate was changed when executing permute in AggregateOnCalcToAggregateUnifyRule, then If this match can't produce result, Visitor can not skip out because the condition "queryDescendant == r.after" is not met. I changed it to use clone of input, when original input would not be changed when this result is null.

junheleo avatar Jun 05 '23 03:06 junheleo

This pull request has been marked as stale due to 30 days of inactivity. It will be closed in 90 days if no further activity occurs. If you think that’s incorrect or this pull request requires a review, please simply write any comment. If closed, you can revive the PR at any time and @mention a reviewer or discuss it on the [email protected] list. Thank you for your contributions.

github-actions[bot] avatar Dec 31 '24 03:12 github-actions[bot]

This pull request has been marked as stale due to 30 days of inactivity. It will be closed in 90 days if no further activity occurs. If you think that’s incorrect or this pull request requires a review, please simply write any comment. If closed, you can revive the PR at any time and @mention a reviewer or discuss it on the [email protected] list. Thank you for your contributions.

github-actions[bot] avatar Mar 21 '25 03:03 github-actions[bot]

This pull request has been marked as stale due to 30 days of inactivity. It will be closed in 90 days if no further activity occurs. If you think that’s incorrect or this pull request requires a review, please simply write any comment. If closed, you can revive the PR at any time and @mention a reviewer or discuss it on the [email protected] list. Thank you for your contributions.

github-actions[bot] avatar Jun 29 '25 03:06 github-actions[bot]

This pull request has been marked as stale due to 30 days of inactivity. It will be closed in 90 days if no further activity occurs. If you think that’s incorrect or this pull request requires a review, please simply write any comment. If closed, you can revive the PR at any time and @mention a reviewer or discuss it on the [email protected] list. Thank you for your contributions.

github-actions[bot] avatar Oct 28 '25 03:10 github-actions[bot]