AfterShip / clickhouse-sql-parser / 25881054565

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

14 of 20 new or added lines in 2 files covered. (70.0%)

1330 existing lines in 3 files now uncovered.

8261 of 15996 relevant lines covered (51.64%)

3009.7 hits per line