FlyingZC commented on PR #38728:
URL: https://github.com/apache/shardingsphere/pull/38728#issuecomment-4552432619
### Decision
- **Merge Verdict: Mergeable**
- **Reviewed Scope:** PR latest head `ca71cd7`; reviewed
`test/it/parser/src/main/resources/case/dml/delete.xml` and
`test/it/parser/src/main/resources/sql/supported/dml/delete.xml`.
- **Not Reviewed Scope:** Full parser matrix beyond the CI result.
- **Need Expert Review:** No.
### Basis
- This PR only adds one MySQL parser IT case for multi-table `DELETE` with
CTE; no production parser logic is changed.
- The syntax is valid for MySQL: `DELETE` can start with `WITH`, and MySQL
supports multi-table `DELETE ... FROM table_references`.
- Local verification passed: `InternalMySQLParserIT` ran `1890` cases with
`0` failures and `0` errors.
- GitHub checks passed: CI, CheckStyle, Spotless, and License are all
green.
### Pre-Merge Checks
- No unrelated file changes found.
- No compatibility or runtime risk found for this test-only change.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
