Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Allow parsing ALTER TABLE statement with column check constraint#554

Merged
MauricioFauth merged 1 commit intophpmyadmin:5.9.xphpmyadmin/sql-parser:5.9.xfrom
MoonE:support-alter-table-with-column-check-constraintMoonE/sql-parser:support-alter-table-with-column-check-constraintCopy head branch name to clipboard
May 23, 2024
Merged

Allow parsing ALTER TABLE statement with column check constraint#554
MauricioFauth merged 1 commit intophpmyadmin:5.9.xphpmyadmin/sql-parser:5.9.xfrom
MoonE:support-alter-table-with-column-check-constraintMoonE/sql-parser:support-alter-table-with-column-check-constraintCopy head branch name to clipboard

Conversation

@MoonE
Copy link
Contributor

@MoonE MoonE commented Apr 18, 2024

CHECK is in Parser::STATEMENT_PARSERS, but also allowed in the column definition.

This doesn't enforce an order for the check constraint. It seems to be only allowed at the end of the column definition, only the column order (FIRST / AFTER x) can follow.

Signed-off-by: Maximilian Krög <maxi_kroeg@web.de>
@williamdes williamdes requested a review from MauricioFauth May 9, 2024 08:18
@MauricioFauth MauricioFauth merged commit f136374 into phpmyadmin:5.9.x May 23, 2024
@MauricioFauth MauricioFauth self-assigned this May 23, 2024
@MoonE MoonE deleted the support-alter-table-with-column-check-constraint branch May 23, 2024 22:06
@williamdes williamdes added this to the 5.10.0 milestone Jun 13, 2024
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.

3 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.