Skip to content

engine: add force index to joins#77

Open
ShimmerGlass wants to merge 1 commit intoclems4ever:masterfrom
ShimmerGlass:force-index
Open

engine: add force index to joins#77
ShimmerGlass wants to merge 1 commit intoclems4ever:masterfrom
ShimmerGlass:force-index

Conversation

@ShimmerGlass
Copy link
Copy Markdown
Collaborator

innodb sometimes selects a query plan order that is not optimized. we
force the indexes used to remedy that

innodb sometimes selects a query plan order that is not optimized. we
force the indexes used to remedy that
Copy link
Copy Markdown
Collaborator

@jmbass jmbass left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM but I'd still like to benchmark this on a future PR.

You removed redundant indexes from the DB in this PR, so we might not need to force them any longer.

Copy link
Copy Markdown
Owner

@clems4ever clems4ever left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I did not know that we could force an index. Nice!

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