[CALCITE-7550] SqlUpdate and SqlDelete unparse EXISTS subqueries without parentheses

[zzwqqq]

Jira Link

CALCITE-7550

Changes Proposed

Fixes unparsing of EXISTS subqueries in UPDATE and DELETE statements.

SqlUpdate and SqlDelete were unparsing the WHERE condition directly, which could produce invalid SQL such as WHERE EXISTS SELECT ....

This change moves the existing WHERE unparsing logic from SqlSelectOperator into SqlUtil.unparseWhereClause and reuses it for SELECT, UPDATE, and DELETE, so DML statements use the same WHERE_LIST frame behavior as queries.

Adds regression tests for UPDATE and DELETE with EXISTS predicates.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
97.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[dssysolyatin]

Is the same problem valid for MERGE ? I think it is possible to specify EXISTS inside ON clause

[mihaibudiu]

@zzwqqq I hope you can answer @dssysolyatin's question before we merge this
Maybe that's the scope for a different issue, but it would be good not to lose this

[zzwqqq]

Is the same problem valid for MERGE ? I think it is possible to specify EXISTS inside ON clause

Thanks for pointing this out. I have not checked the MERGE case in detail yet, but based on your comment there may be a similar issue.

I opened CALCITE-7585 to track this separately, since it seems related but outside the current UPDATE/DELETE fix. I will follow up there with a concrete reproduction and more details once I confirm the behavior. If the issue is valid, I will continue working on it there.

[zzwqqq]

@zzwqqq I hope you can answer @dssysolyatin's question before we merge this Maybe that's the scope for a different issue, but it would be good not to lose this

Thanks. I opened CALCITE-7585 to track this separately. I will try to reproduce the MERGE case there and add more details once the behavior is confirmed.