Skip to content

[CALCITE-7928] Volcano optimizer takes a very long time for a cross join lateral query - #5410

Open
mihaibudiu wants to merge 1 commit into
apache:mainfrom
mihaibudiu:issue7928
Open

mihaibudiu wants to merge 1 commit into
apache:mainfrom
mihaibudiu:issue7928

Conversation

@mihaibudiu

Copy link
Copy Markdown
Contributor

Jira Link

CALCITE-7928

Changes Proposed

I have re-enabled the query but disabled (for this test only) a Volcano rule which is executed about 2 million times for optimizing the program generated by the decorrelator. With this change optimization is much faster.

@mihaibudiu

Copy link
Copy Markdown
Contributor Author

Solves the same problem as #5408.
That PR changes the decorrelator. Maybe these changes are actually useful, but the underlying problem is not really the decorrelator.

@sonarqubecloud

Copy link
Copy Markdown

@xiedeyantu

Copy link
Copy Markdown
Member

I agree; I’ve also found this to be the reason, and I think this change is reasonable. However, should we keep this JIRA ticket open, or should we create a new one?

@rubenada

Copy link
Copy Markdown
Contributor

IMO we should merge this patch, but as a "Minor: ..." commit, unrelated to CALCITE-7928 because it does not address the root cause of the planning time explosion.

@mihaibudiu

Copy link
Copy Markdown
Contributor Author

I will resubmit the test fix as a separate PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants