Merge pull request #1729 from taozhi8833998/fix-alter-table-modify-co…

…lumn-mysql fix: alter table modify column keyword is optional in mysql
taozhi8833998 · Dec 20, 2023 · 8de34cb · 8de34cb
2 parents 239bdcd + a3bf84d
commit 8de34cb
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 2 deletions.
diff --git a/pegjs/mariadb.pegjs b/pegjs/mariadb.pegjs
@@ -875,9 +875,11 @@ ALTER_ADD_COLUMN
 
 ALTER_MODIFY_COLUMN
   = KW_MODIFY __
+    kc:KW_COLUMN? __
     cd:create_column_definition {
       return {
         action: 'modify',
+        keyword: kc,
         ...cd,
         resource: 'column',
         type: 'alter',

diff --git a/pegjs/mysql.pegjs b/pegjs/mysql.pegjs
@@ -1076,9 +1076,11 @@ ALTER_ADD_COLUMN
 
 ALTER_MODIFY_COLUMN
   = KW_MODIFY __
+    kc:KW_COLUMN? __
     cd:create_column_definition {
       return {
         action: 'modify',
+        keyword: kc,
         ...cd,
         resource: 'column',
         type: 'alter',

diff --git a/test/mysql-mariadb.spec.js b/test/mysql-mariadb.spec.js
@@ -716,8 +716,8 @@ describe('mysql', () => {
       {
         title: 'alter table modify column',
         sql: [
-          "ALTER TABLE gifshow.reporter MODIFY update_at BIGINT UNSIGNED NOT NULL COMMENT 'update_at';",
-          "ALTER TABLE `gifshow`.`reporter` MODIFY `update_at` BIGINT UNSIGNED NOT NULL COMMENT 'update_at'"
+          "ALTER TABLE gifshow.reporter MODIFY Column update_at BIGINT UNSIGNED NOT NULL COMMENT 'update_at';",
+          "ALTER TABLE `gifshow`.`reporter` MODIFY COLUMN `update_at` BIGINT UNSIGNED NOT NULL COMMENT 'update_at'"
         ]
       },
       {