From 6523dabcf886c69be58ca8e486683fb6e02e0f16 Mon Sep 17 00:00:00 2001
From: Ramnivas Laddad <ramnivas@exograph.dev>
Date: Thu, 19 Dec 2024 01:10:53 -0800
Subject: [PATCH] Allow foreign table constraint without columns (#1608)

---
 src/ast/ddl.rs            | 6 ++++--
 src/parser/mod.rs         | 2 +-
 tests/sqlparser_common.rs | 1 +
 3 files changed, 6 insertions(+), 3 deletions(-)

diff --git a/src/ast/ddl.rs b/src/ast/ddl.rs
index 849b583ed..c87960679 100644
--- a/src/ast/ddl.rs
+++ b/src/ast/ddl.rs
@@ -885,12 +885,14 @@ impl fmt::Display for TableConstraint {
             } => {
                 write!(
                     f,
-                    "{}FOREIGN KEY ({}) REFERENCES {}({})",
+                    "{}FOREIGN KEY ({}) REFERENCES {}",
                     display_constraint_name(name),
                     display_comma_separated(columns),
                     foreign_table,
-                    display_comma_separated(referred_columns),
                 )?;
+                if !referred_columns.is_empty() {
+                    write!(f, "({})", display_comma_separated(referred_columns))?;
+                }
                 if let Some(action) = on_delete {
                     write!(f, " ON DELETE {action}")?;
                 }
diff --git a/src/parser/mod.rs b/src/parser/mod.rs
index c0aa0acba..ae44de1b5 100644
--- a/src/parser/mod.rs
+++ b/src/parser/mod.rs
@@ -6830,7 +6830,7 @@ impl<'a> Parser<'a> {
                 let columns = self.parse_parenthesized_column_list(Mandatory, false)?;
                 self.expect_keyword(Keyword::REFERENCES)?;
                 let foreign_table = self.parse_object_name(false)?;
-                let referred_columns = self.parse_parenthesized_column_list(Mandatory, false)?;
+                let referred_columns = self.parse_parenthesized_column_list(Optional, false)?;
                 let mut on_delete = None;
                 let mut on_update = None;
                 loop {
diff --git a/tests/sqlparser_common.rs b/tests/sqlparser_common.rs
index 1bf9383af..d04066c70 100644
--- a/tests/sqlparser_common.rs
+++ b/tests/sqlparser_common.rs
@@ -4167,6 +4167,7 @@ fn parse_alter_table_constraints() {
     check_one("UNIQUE (id)");
     check_one("FOREIGN KEY (foo, bar) REFERENCES AnotherTable(foo, bar)");
     check_one("CHECK (end_date > start_date OR end_date IS NULL)");
+    check_one("CONSTRAINT fk FOREIGN KEY (lng) REFERENCES othertable4");
 
     fn check_one(constraint_text: &str) {
         match alter_table_op(verified_stmt(&format!(