Skip to content

Conversation

@mamcx
Copy link
Contributor

@mamcx mamcx commented Feb 4, 2025

Description of Changes

In the case of permutations, multi-column indexes were not used when optimizing the plan, the pass assumed the order of the columns, and now it looks for it without this assumption.

Expected complexity level and risk

1

Testing

  • Addd unit test that check permuation for 2/3 columns

@mamcx mamcx added bugfix Fixes something that was expected to work differently release-1.0 labels Feb 4, 2025
@mamcx mamcx self-assigned this Feb 4, 2025
@mamcx mamcx force-pushed the mamcx/fix-multi-column-idx branch from 3236bb1 to 99700d7 Compare February 5, 2025 16:23
@mamcx mamcx force-pushed the mamcx/fix-multi-column-idx branch from 99700d7 to 3fcd754 Compare February 5, 2025 20:16
@mamcx mamcx added this pull request to the merge queue Feb 6, 2025
Merged via the queue into master with commit 02a61b2 Feb 6, 2025
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix Fixes something that was expected to work differently release-1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants