Skip to content

(RC2) Fix ESCAPE clause for Azure Synapse. (#34463)#34508

Closed
cincuranet wants to merge 1 commit intodotnet:release/9.0from
cincuranet:backport-synapse-escape
Closed

(RC2) Fix ESCAPE clause for Azure Synapse. (#34463)#34508
cincuranet wants to merge 1 commit intodotnet:release/9.0from
cincuranet:backport-synapse-escape

Conversation

@cincuranet
Copy link
Copy Markdown
Contributor

@cincuranet cincuranet commented Aug 22, 2024

Port of #34463

Fixes #33555

Description

Fixes translation of Contains/StartsWith/EndsWith into SQL on Azure Synapse.

Customer impact

Customers using Azure Synapse are unable to run queries using Contains/StartsWith/EndsWith because improper SQL is generated.

How found

Reported by customer.

Regression

Yes.

Testing

Tests added.

Risk

Low.

@cincuranet cincuranet requested review from a team and artl93 August 22, 2024 10:11
@roji
Copy link
Copy Markdown
Member

roji commented Aug 22, 2024

@cincuranet can you please fill out the template and ping @artl93 as this is for RC2?

@cincuranet
Copy link
Copy Markdown
Contributor Author

@roji
Copy link
Copy Markdown
Member

roji commented Aug 22, 2024

You can copy-paste on from one of the PRs that have already been merged to release/9.0, e.g. #34495.

@cincuranet cincuranet changed the title [release/9.0] Fix ESCAPE clause for Azure Synapse. (#34463) (RC2) Fix ESCAPE clause for Azure Synapse. (#34463) Aug 22, 2024
@cincuranet
Copy link
Copy Markdown
Contributor Author

Done. @artl93 was already added as reviewer.

@cincuranet
Copy link
Copy Markdown
Contributor Author

Closing in favor of #34509.

@cincuranet cincuranet closed this Aug 22, 2024
@cincuranet cincuranet deleted the backport-synapse-escape branch August 22, 2024 12:02
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