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.

Sign up for GitHub

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

Jump to bottom

Change supportsLikeEscapeClause to check for AzureDW #2092

Merged
merged 1 commit into from
Mar 24, 2023
Merged

Change supportsLikeEscapeClause to check for AzureDW #2092

merged 1 commit into from
Mar 24, 2023

Conversation

Jeffery-Wasty
Copy link
Member

Change `supportsLikeEscapeClause() to return false if server is Azure DW, as ESCAPE clause is not supported there (https://learn.microsoft.com/en-us/sql/t-sql/language-elements/like-transact-sql?view=sql-server-ver16).

@Jeffery-Wasty Jeffery-Wasty linked an issue Mar 15, 2023 that may be closed by this pull request
DatabaseMetaData.supportsLikeEscapeClause() Returns true for Synapse Despite the ESCAPE Clause Not Being Supported #2091
Closed
@Jeffery-Wasty Jeffery-Wasty added this to In progress in MSSQL JDBC via automation Mar 17, 2023
@Jeffery-Wasty Jeffery-Wasty moved this from In progress to Under Peer Review in MSSQL JDBC Mar 17, 2023
@lilgreenbird lilgreenbird added this to the 12.3.0 milestone Mar 24, 2023
@Jeffery-Wasty Jeffery-Wasty merged commit b0a7c6f into main Mar 24, 2023
MSSQL JDBC automation moved this from Under Peer Review to Closed/Merged PRs Mar 24, 2023
@Jeffery-Wasty Jeffery-Wasty deleted the supportsEscapeFix branch March 28, 2023 19:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lilgreenbird lilgreenbird lilgreenbird approved these changes

@tkyc tkyc tkyc approved these changes

Assignees
No one assigned
Labels
None yet
Projects
MSSQL JDBC
  
Closed/Merged PRs
Milestone
12.3.0
3 participants
@Jeffery-Wasty @lilgreenbird @tkyc