Skip to content

Fix STRAIGHT_JOIN constraint when table alias is absent#1812

Merged
iffyio merged 2 commits intoapache:mainfrom
killertux:fix_straight_join_as_alias_for_mysql
Apr 15, 2025
Merged

Fix STRAIGHT_JOIN constraint when table alias is absent#1812
iffyio merged 2 commits intoapache:mainfrom
killertux:fix_straight_join_as_alias_for_mysql

Conversation

@killertux
Copy link
Copy Markdown
Contributor

In #1802 was added support for MySQL STRAIGHT_JOIN, but it was not added as a RESERVED_FOR_TABLE_ALIAS. Because of this, it fails to parse queries where we are not using an alias.

Copy link
Copy Markdown
Contributor

@iffyio iffyio left a comment

Choose a reason for hiding this comment

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

Thanks @killertux! Left one minor comment to merge the test cases otherwise this looks good to me

Comment thread tests/sqlparser_mysql.rs
Co-authored-by: Ifeanyi Ubah <ify1992@yahoo.com>
Copy link
Copy Markdown
Contributor

@iffyio iffyio left a comment

Choose a reason for hiding this comment

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

Thanks @killertux!
cc @alamb

@iffyio iffyio changed the title Fix STRAIGHT_JOIN not in RESERVED_FOR_TABLE_ALIAS Fix STRAIGHT_JOIN constraint when table alias is absent Apr 15, 2025
@iffyio iffyio merged commit 81d8909 into apache:main Apr 15, 2025
9 checks passed
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.

2 participants