Allow trailing comma in CREATE TABLE

At least MSSQL supports it, not sure about others.

Author: nickolay

src/sqlparser.rs

@@ -795,17 +795,12 @@ impl Parser {
             } else {
                 return self.expected("column name or constraint definition", self.peek_token());
             }
-            match self.next_token() {
-                Some(Token::Comma) => {}
-                Some(Token::RParen) => {
-                    break;
-                }
-                other => {
-                    return parser_err!(format!(
-                        "Expected ',' or ')' after column definition but found {:?}",
-                        other
-                    ));
-                }
+            let comma = self.consume_token(&Token::Comma);
+            if self.consume_token(&Token::RParen) {
+                // allow a trailing comma, even though it's not in standard
+                break;
+            } else if !comma {
+                return self.expected("',' or ')' after column definition", self.peek_token());
             }
         }
 

tests/sqlparser_common.rs

@@ -573,6 +573,12 @@ fn parse_create_table() {
     }
 }
 
+#[test]
+fn parse_create_table_trailing_comma() {
+    let sql = "CREATE TABLE foo (bar int,)";
+    all_dialects().one_statement_parses_to(sql, "CREATE TABLE foo (bar int)");
+}
+
 #[test]
 fn parse_create_external_table() {
     let sql = "CREATE EXTERNAL TABLE uk_cities (\