Skip to content

Add support for string literal concatenation#2003

Merged
iffyio merged 3 commits intoapache:mainfrom
SatoriCyber:adjacent_string_literal_concatenation-support
Sep 2, 2025
Merged

Add support for string literal concatenation#2003
iffyio merged 3 commits intoapache:mainfrom
SatoriCyber:adjacent_string_literal_concatenation-support

Conversation

@etgarperets
Copy link
Copy Markdown
Contributor

… space in mySQL

Comment thread src/dialect/mysql.rs Outdated
Comment thread src/dialect/mod.rs Outdated
Comment thread src/parser/mod.rs Outdated
Comment thread tests/sqlparser_mysql.rs Outdated
@iffyio iffyio changed the title SGA-3801 Added support for concatenating string literals saperated by… Added support for string literal concatenation Aug 20, 2025
@iffyio iffyio changed the title Added support for string literal concatenation Add support for string literal concatenation Aug 20, 2025
@etgarperets etgarperets force-pushed the adjacent_string_literal_concatenation-support branch from 04c8f9b to ecdee10 Compare August 26, 2025 07:53
@etgarperets
Copy link
Copy Markdown
Contributor Author

@iffyio All fixes are in and waiting on a review

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 @etgarperets! Changes look good to me overall, left a minor comment

Comment thread src/dialect/mod.rs Outdated
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.

LGTM! Thanks @etgarperets!
cc @alamb

@iffyio iffyio merged commit cff2833 into apache:main Sep 2, 2025
10 checks passed
@etgarperets etgarperets deleted the adjacent_string_literal_concatenation-support branch September 2, 2025 11:43
ayman-sigma pushed a commit to sigmacomputing/sqlparser-rs that referenced this pull request Feb 3, 2026
Co-authored-by: Ifeanyi Ubah <ify1992@yahoo.com>
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