Skip to content

Model MySQL CREATE TRIGGER statements - #2548

Merged
manticore-projects merged 1 commit into
JSQLParser:masterfrom
minleejae:feat/mysql-create-trigger
Sep 5, 2026
Merged

Model MySQL CREATE TRIGGER statements#2548
manticore-projects merged 1 commit into
JSQLParser:masterfrom
minleejae:feat/mysql-create-trigger

Conversation

@minleejae

Copy link
Copy Markdown
Contributor

Summary

  • replace the unsupported fallback for MySQL CREATE TRIGGER with a dedicated statement AST
  • expose definer accounts, trigger timing/event, target table, ordering, and body as structured values
  • retain trigger bodies as normal statement nodes so callers can inspect assignments and referenced tables directly

Validation

  • ./gradlew spotlessApply test spotlessCheck checkstyleMain checkstyleTest
  • executed before/after, insert/update/delete, definer, and ordered-trigger forms against MySQL 8.4

@manticore-projects

Copy link
Copy Markdown
Contributor

Please resolve conflicts.

@minleejae
minleejae force-pushed the feat/mysql-create-trigger branch 2 times, most recently from 9f57828 to 5b24560 Compare September 5, 2026 00:46
@minleejae

minleejae commented Sep 5, 2026

Copy link
Copy Markdown
Contributor Author

@manticore-projects The merge conflicts have been resolved, and the branch has been rebased onto the latest master. This PR is ready for review again. Thank you.

@minleejae
minleejae force-pushed the feat/mysql-create-trigger branch from 5b24560 to 6329af6 Compare September 5, 2026 01:55
@manticore-projects

Copy link
Copy Markdown
Contributor

Give it one last push please!

@minleejae
minleejae force-pushed the feat/mysql-create-trigger branch from 6329af6 to c2b5a6a Compare September 5, 2026 01:59
@manticore-projects
manticore-projects merged commit 03b8fc4 into JSQLParser:master Sep 5, 2026
3 checks passed
@manticore-projects

Copy link
Copy Markdown
Contributor

@minleejae: Profound Thank You! for your work and effort! I trust the MySQL will appreciate you! Because the JSQLParser community does, a lot!

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