Skip to content

PRIMARY in index hint list#8158

Merged
systay merged 2 commits intovitessio:release-9.0from
planetscale:use-index-primary
May 20, 2021
Merged

PRIMARY in index hint list#8158
systay merged 2 commits intovitessio:release-9.0from
planetscale:use-index-primary

Conversation

@GuptaManan100
Copy link
Contributor

Description

This PR fixes the regression created in release 9.0 due to moving PRIMARY from non-reserved to reserved keywords

  • Vitess 8.0

    > SELECT id FROM blog_posts USE INDEX (PRIMARY) WHERE id = 10;
    +------+
    |   id |
    |------|
    |   10 |
    +------+
    1 row in set
    Time: 0.004s
    
  • Vitess 9.0

    > SELECT id FROM blog_posts USE INDEX (PRIMARY) WHERE id = 10;
    (1105, "vtgate: http://vtgate:15000/: syntax error at position 67 near 'PRIMARY'")
    

Related Issue(s)

Fixes #8148

Checklist

  • Tests were added or are not required
  • Documentation was added or is not required

Signed-off-by: GuptaManan100 <manan@planetscale.com>
Signed-off-by: GuptaManan100 <manan@planetscale.com>
@systay systay merged commit 7cb2b70 into vitessio:release-9.0 May 20, 2021
@systay systay deleted the use-index-primary branch May 20, 2021 11:32
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.

2 participants