Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

sql: support FORCE_INDEX and NO_INDEX_JOIN hints #5806

Merged

Conversation

RaduBerinde
Copy link
Member

Add support for SELECT FROM table@{FORCE_INDEX=idx,NO_INDEX_JOIN}, as
described in the index_hints RFC.

Closes #5625.


This change is Reviewable

@petermattis
Copy link
Collaborator

:lgtm:


Review status: 0 of 13 files reviewed at latest revision, 2 unresolved discussions.


sql/index_selection.go, line 179 [r1] (raw file):
s/util.Errorf/fmt.Errorf/g as this is a user facing error.


sql/parser/sql.y, line 3630 [r1] (raw file):
Should @{FORCE_INDEX=bar,FORCE_INDEX=foo} be an error? Looks like it will currently work.


Comments from the review on Reviewable.io

Add support for `SELECT FROM table@{FORCE_INDEX=idx,NO_INDEX_JOIN}`, as
described in the index_hints RFC.

Closes cockroachdb#5625.
@RaduBerinde RaduBerinde force-pushed the index-hint-strikes-back branch from bc57d75 to 8bcd67a Compare April 1, 2016 18:14
@RaduBerinde
Copy link
Member Author

TFTR! Updated.


Review status: 0 of 13 files reviewed at latest revision, 2 unresolved discussions.


Comments from the review on Reviewable.io

@petermattis
Copy link
Collaborator

:lgtm:


Review status: 0 of 13 files reviewed at latest revision, 2 unresolved discussions.


Comments from the review on Reviewable.io

@RaduBerinde RaduBerinde merged commit d3fd734 into cockroachdb:master Apr 1, 2016
@RaduBerinde RaduBerinde deleted the index-hint-strikes-back branch April 1, 2016 19:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants