fix: FromItem with Alias without AS keyword

- `VALUES ... T(x,y)`

Signed-off-by: Andreas Reichel <andreas@manticore-projects.com>

Author: manticore-projects

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

@@ -2650,7 +2650,7 @@ Alias Alias():
 {
     (
         LOOKAHEAD(3) (
-            // Aliases with Columns:
+            // Aliases with AS and Columns, but optional identifier:
             //      SELECT fun(x) AS (a,b,c)
             //      SELECT fun(x) AS T(a,b,c)
             <K_AS>
@@ -2667,23 +2667,20 @@ Alias Alias():
         )
         |
         (
-            // Aliases without Columns:
+            // Aliases with identifier but optional AS and Columns:
             //      SELECT fun(x) AS T
             //      SELECT fun(x) T
+            //      SELECT fun(x) T(a,b,c)
 
             [<K_AS> { useAs = true; } ]
             ( name=RelObjectNameWithoutStart() | token=<S_CHAR_LITERAL> { name=token.image; } )
             { alias = new Alias(name,useAs); }
 
-            /* Should not be allowed/needed
-               We can bring this back only if people complain
-
             [ LOOKAHEAD(2) "(" { List<Alias.AliasColumn> list = new ArrayList<Alias.AliasColumn>(); }
                 colname = RelObjectName() [ colDataType = ColDataType() ] { list.add(new Alias.AliasColumn(colname, colDataType)); }
                 ("," { colDataType=null; } colname = RelObjectName() [ colDataType = ColDataType()] { list.add(new Alias.AliasColumn(colname, colDataType)); } )*
             ")" { alias.setAliasColumns(list); } ]
 
-            */
         )
     )
     { return alias; }

src/test/java/net/sf/jsqlparser/statement/values/ValuesTest.java

@@ -64,4 +64,11 @@ public void testObject() {
 
         valuesStatement.accept(new StatementVisitorAdapter());
     }
+
+    @Test
+    public void testValuesWithAliasWithoutAs() throws JSQLParserException {
+        String sqlStr = "SELECT a, b, cume_dist() OVER (PARTITION BY a ORDER BY b) AS cume_dist\n" +
+                "    FROM VALUES ('A1', 2), ('A1', 1), ('A2', 3), ('A1', 1) tab(a, b);";
+        assertSqlCanBeParsedAndDeparsed(sqlStr, true);
+    }
 }