Skip to content

Commit 3623d50

Browse files
authored
Revert "Allow empty options for BigQuery (apache#1657)"
This reverts commit 762789b.
1 parent bed9604 commit 3623d50

File tree

2 files changed

+1
-7
lines changed

2 files changed

+1
-7
lines changed

src/parser/mod.rs

+1-1
Original file line numberDiff line numberDiff line change
@@ -7341,7 +7341,7 @@ impl<'a> Parser<'a> {
73417341
pub fn parse_options(&mut self, keyword: Keyword) -> Result<Vec<SqlOption>, ParserError> {
73427342
if self.parse_keyword(keyword) {
73437343
self.expect_token(&Token::LParen)?;
7344-
let options = self.parse_comma_separated0(Parser::parse_sql_option, Token::RParen)?;
7344+
let options = self.parse_comma_separated(Parser::parse_sql_option)?;
73457345
self.expect_token(&Token::RParen)?;
73467346
Ok(options)
73477347
} else {

tests/sqlparser_bigquery.rs

-6
Original file line numberDiff line numberDiff line change
@@ -473,12 +473,6 @@ fn parse_create_table_with_options() {
473473
r#"description = "table option description")"#
474474
);
475475
bigquery().verified_stmt(sql);
476-
477-
let sql = "CREATE TABLE foo (x INT64) OPTIONS()";
478-
bigquery().verified_stmt(sql);
479-
480-
let sql = "CREATE TABLE db.schema.test (x INT64 OPTIONS(description = 'An optional INTEGER field')) OPTIONS()";
481-
bigquery().verified_stmt(sql);
482476
}
483477

484478
#[test]

0 commit comments

Comments
 (0)