#1516 rename without column keyword (#1533)

* Adjust Gradle to JUnit 5

Parallel Test execution
Gradle Caching
Explicitly request for latest JavaCC 7.0.10

* Do not mark SpeedTest for concurrent execution

* Remove unused imports

* Adjust Gradle to JUnit 5

Parallel Test execution
Gradle Caching
Explicitly request for latest JavaCC 7.0.10

* Do not mark SpeedTest for concurrent execution

* Remove unused imports

* `RENAME ... TO ...` without `COLUMN` keyword

Fixes #1516

Author: manticore-projects

src/main/jjtree/net/sf/jsqlparser/parser/JSqlParserCC.jjt

@@ -5724,13 +5724,9 @@ AlterExpression AlterExpression():
           )
       )
       |
-      (<K_CHANGE>
-        {
-          alterExp.setOperation(AlterOperation.CHANGE);
-        }
-        (
-          <K_COLUMN> { alterExp.hasColumn(true); alterExp.setOptionalSpecifier("COLUMN"); } | {}
-        )
+      (
+        <K_CHANGE> { alterExp.setOperation(AlterOperation.CHANGE); }
+        [ <K_COLUMN> { alterExp.hasColumn(true); alterExp.setOptionalSpecifier("COLUMN"); } ]
         (
           (tk=<S_IDENTIFIER> | tk=<S_QUOTED_IDENTIFIER>)
           alterExpressionColumnDataType = AlterExpressionColumnDataType() {  alterExp.withColumnOldName(tk.image).addColDataType(alterExpressionColumnDataType); }
@@ -5806,7 +5802,7 @@ AlterExpression AlterExpression():
          sk3 = RelObjectName() {alterExp.addParameters(sk3); }
       )
       |
-      LOOKAHEAD(2)  <K_RENAME> <K_COLUMN> { alterExp.setOperation(AlterOperation.RENAME); alterExp.hasColumn(true);}
+      LOOKAHEAD(2)   <K_RENAME> { alterExp.setOperation(AlterOperation.RENAME); } [ <K_COLUMN> {  alterExp.hasColumn(true);} ]
                     ( tk=<S_IDENTIFIER>    | tk=<S_QUOTED_IDENTIFIER> ) { alterExp.setColOldName(tk.image); }
                     <K_TO>
                     (tk2=<S_IDENTIFIER>    | tk2=<S_QUOTED_IDENTIFIER>) { alterExp.setColumnName(tk2.image); }

src/test/java/net/sf/jsqlparser/statement/alter/AlterTest.java

@@ -471,6 +471,7 @@ public void testAlterTableAlterColumnDropNotNullIssue918() throws JSQLParserExce
 
     @Test
     public void testAlterTableRenameColumn() throws JSQLParserException {
+        // With Column Keyword
         String sql = "ALTER TABLE \"test_table\" RENAME COLUMN \"test_column\" TO \"test_c\"";
         assertSqlCanBeParsedAndDeparsed(sql);
 
@@ -479,6 +480,10 @@ public void testAlterTableRenameColumn() throws JSQLParserException {
         assertEquals(expression.getOperation(), AlterOperation.RENAME);
         assertEquals(expression.getColOldName(), "\"test_column\"");
         assertEquals(expression.getColumnName(), "\"test_c\"");
+
+        // Without Column Keyword
+        sql = "ALTER TABLE \"test_table\" RENAME \"test_column\" TO \"test_c\"";
+        assertSqlCanBeParsedAndDeparsed(sql);
     }
 
     @Test