Skip to content

fix(parser): model MySQL column type modifiers - #2537

Merged
manticore-projects merged 3 commits into
JSQLParser:masterfrom
minleejae:fix/mysql-column-type-modifiers
Sep 4, 2026
Merged

fix(parser): model MySQL column type modifiers#2537
manticore-projects merged 3 commits into
JSQLParser:masterfrom
minleejae:fix/mysql-column-type-modifiers

Conversation

@minleejae

Copy link
Copy Markdown
Contributor

Summary

  • add structured SIGNED and UNSIGNED state to ColDataType
  • model ZEROFILL consistently with and without numeric type arguments
  • keep existing CAST(... AS UNSIGNED) forms working
  • prevent column type modifiers from leaking into ColumnDefinition.columnSpecs

Testing

  • ./gradlew check
  • includes regression coverage for numeric column definitions and unsigned array casts

Fixes #2532

AI assistance

OpenAI Codex assisted with the implementation and test coverage. The changes were manually reviewed and verified against MySQL syntax.

@manticore-projects

Copy link
Copy Markdown
Contributor

Please resolve conflicts from overlapping PRs.

@minleejae

Copy link
Copy Markdown
Contributor Author

@manticore-projects Conflicts have been resolved, and all CI checks are passing.

@manticore-projects

Copy link
Copy Markdown
Contributor

Merge conflicts, sorry.

@manticore-projects
manticore-projects merged commit bb2f269 into JSQLParser:master Sep 4, 2026
5 of 6 checks passed
@manticore-projects

Copy link
Copy Markdown
Contributor

Thank you!

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.

[BUG] 5.4-SNAPSHOT: MySQL numeric column modifiers are inconsistently modeled

2 participants