AfterShip / clickhouse-sql-parser / 25881230625

52%

master: 52%

Build Type

Pull #270

github

Committed by sharadgaur

Commit Message

fix: support column list before ENGINE in materialized view

ClickHouse SHOW CREATE TABLE for refreshable materialized views with
ENGINE (e.g. Memory) outputs the column list between REFRESH and ENGINE:

  CREATE MATERIALIZED VIEW db1.mv_name
  REFRESH EVERY 1 SECOND
  (
      col1 String,
      col2 Int8
  )
  ENGINE = Memory
  AS SELECT ...

The parser previously expected TO or ENGINE immediately after REFRESH
clauses. This adds support for the (columns) before ENGINE pattern by
parsing a TableSchemaClause when a left paren is encountered.

Added TableSchema field to CreateMaterializedView AST node and updated
FormatSQL to emit it before ENGINE.

Pull Request Pull Request #270: fix: support column list before ENGINE in CREATE MATERIALIZED VIEW

Coverage Stats

15 of 23 new or added lines in 3 files covered. (65.22%)

1169 existing lines in 2 files now uncovered.

8262 of 15999 relevant lines covered (51.64%)

3009.2 hits per line