Skip to content

Fix #8109: Plan/Performance regression when using special construct for IN#8425

Merged
dyemanov merged 2 commits intov5.0-releasefrom
work/gh-8109
Feb 3, 2025
Merged

Fix #8109: Plan/Performance regression when using special construct for IN#8425
dyemanov merged 2 commits intov5.0-releasefrom
work/gh-8109

Conversation

@dyemanov
Copy link
Copy Markdown
Member

@dyemanov dyemanov commented Feb 1, 2025

The solution is to decompose all list items containing field references inside into separate ORed equalities, so they could be optimized separately.

@dyemanov dyemanov requested a review from hvlad February 1, 2025 15:57
@dyemanov dyemanov self-assigned this Feb 1, 2025
@dyemanov dyemanov merged commit fd31e52 into v5.0-release Feb 3, 2025
@dyemanov dyemanov deleted the work/gh-8109 branch February 3, 2025 10:10
dyemanov added a commit that referenced this pull request Feb 3, 2025
…or IN (#8425)

* Fix #8109: Plan/Performance regression when using special construct for IN

* Optimize the code a bit: use one loop instead of two loops
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.

Plan/Performance regression when using special construct for IN in FB5.x compared to FB3.x

2 participants