Skip to content

Commit b0df562

Browse files
resolve comments again
1 parent f709f06 commit b0df562

File tree

4 files changed

+9
-2
lines changed

4 files changed

+9
-2
lines changed

src/dialect/clickhouse.rs

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -50,4 +50,8 @@ impl Dialect for ClickHouseDialect {
5050
fn supports_limit_comma(&self) -> bool {
5151
true
5252
}
53+
54+
fn supports_table_function_in_insertion(&self) -> bool {
55+
true
56+
}
5357
}

src/dialect/mod.rs

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -780,6 +780,8 @@ pub trait Dialect: Debug + Any {
780780
///
781781
/// MySQL: <https://dev.mysql.com/doc/refman/8.4/en/insert.html>
782782
fn supports_insert_set(&self) -> bool {
783+
fn supports_table_function_in_insertion(&self) -> bool {
784+
/// Does the dialect support table function in insertion?
783785
false
784786
}
785787
}

src/parser/mod.rs

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8861,7 +8861,9 @@ impl<'a> Parser<'a> {
88618861
/// Parse a table object for insetion
88628862
/// e.g. `some_database.some_table` or `FUNCTION some_table_func(...)`
88638863
pub fn parse_table_object(&mut self) -> Result<TableObject, ParserError> {
8864-
if dialect_of!(self is ClickHouseDialect) && self.parse_keyword(Keyword::FUNCTION) {
8864+
if self.dialect.supports_table_function_in_insertion()
8865+
&& self.parse_keyword(Keyword::FUNCTION)
8866+
{
88658867
let fn_name = self.parse_object_name(false)?;
88668868
self.parse_function_call(fn_name)
88678869
.map(TableObject::TableFunction)

src/test_utils.rs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -154,7 +154,6 @@ impl TestedDialects {
154154
pub fn one_statement_parses_to(&self, sql: &str, canonical: &str) -> Statement {
155155
let mut statements = self.parse_sql_statements(sql).expect(sql);
156156
assert_eq!(statements.len(), 1);
157-
println!("sql: {}", sql);
158157
if !canonical.is_empty() && sql != canonical {
159158
assert_eq!(self.parse_sql_statements(canonical).unwrap(), statements);
160159
}

0 commit comments

Comments
 (0)