Skip to content

fix(dolphin): preserve NOT EXISTS expressions#4526

Open
muratmirgun wants to merge 1 commit into
sqlc-dev:mainfrom
muratmirgun:claude/fix-mysql-not-exists-4512
Open

fix(dolphin): preserve NOT EXISTS expressions#4526
muratmirgun wants to merge 1 commit into
sqlc-dev:mainfrom
muratmirgun:claude/fix-mysql-not-exists-4512

Conversation

@muratmirgun

Copy link
Copy Markdown

Fix MySQL AST conversion dropping NOT from NOT EXISTS expressions.

the affected syntax in two different ways:

  • NOT (EXISTS (...)) as a unary NOT operation
  • NOT EXISTS (...) through ExistsSubqueryExpr.Not

The Dolphin converter did not handle either representation correctly. In the parenthesized case, the entire WHERE expression could be dropped, including nested sqlc.arg parameters. In the direct case, the query was preserved but the NOT operator was lost.

This change:

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.

1 participant