Key Value Options: add support for trailing semicolon

Author: yoavcloud

src/parser/mod.rs

@@ -18409,7 +18409,7 @@ impl<'a> Parser<'a> {
                         return self.expected(" another option or EOF", self.peek_token());
                     }
                 }
-                Token::EOF => break,
+                Token::EOF | Token::SemiColon => break,
                 Token::Comma => {
                     delimiter = KeyValueOptionsDelimiter::Comma;
                     continue;
@@ -18421,7 +18421,7 @@ impl<'a> Parser<'a> {
                     self.prev_token();
                     break;
                 }
-                _ => return self.expected("another option, EOF, Comma or ')'", self.peek_token()),
+                _ => return self.expected("another option, EOF, SemiColon, Comma or ')'", self.peek_token()),
             };
         }
 
@@ -19306,4 +19306,11 @@ mod tests {
             assert!(Parser::parse_sql(&GenericDialect, &sql).is_err());
         }
     }
+
+    #[test]
+    fn test_key_value_options_trailing_semicolon() {
+        let sql = "CREATE USER u1 option1 = 'value1' option2 = 'value2';";
+        let ast = Parser::parse_sql(&GenericDialect, sql);
+        assert!(ast.is_ok());
+    }
 }