AfterShip / clickhouse-sql-parser
52%

DEFAULT BRANCH: master
Repo Added
Token F4x2H6fmkWoZPkof5HTWcZ5HSIXKiqDLa regen
Build 468 Last
Files 15
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

LAST BUILD ON BRANCH master
branch: master
CHANGE BRANCH
x
Sync Branches

coverage: 51.662% (+0.04%) from 51.621%
25913331789

push

github

web-flow 
Fix should allow column list before ENGINE and fix DEPENDS ON multi-table in CREATE MATERIALIZED VIEW (#270)

1. Column list before ENGINE (new):
   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. Added support for (columns) before ENGINE by parsing a
   TableSchemaClause when a left paren is encountered.

2. DEPENDS ON multi-table (bug fix):
   The comma in 'DEPENDS ON db1.mv_a, db1.mv_b' was not consumed before
   parsing the next table identifier. Added missing consumeToken() call
   in the comma loop, matching the pattern used in parser_query.go.

Changes:
- parser/parser_view.go: handle (columns) before ENGINE; consume comma in DEPENDS ON loop
- parser/ast.go: add TableSchema field to CreateMaterializedView
- parser/format.go: emit TableSchema before ENGINE in FormatSQL
- parser/walk.go: traverse TableSchema in Walk

Tests:
- create_materialized_view_rmv_engine_with_columns.sql (column list + ENGINE)
- create_materialized_view_rmv_depends_on_multi.sql (multi-table DEPENDS ON)
- All 21 MV syntax variants tested, all existing tests pass

16 of 28 new or added lines in 4 files covered. (57.14%)

8268 of 16004 relevant lines covered (51.66%)

3015.1 hits per line

Relevant lines Covered
16004 RELEVANT LINES 8268 COVERED LINES
3015.1 HITS PER LINE
Source Files on master

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
25913331789 master Fix should allow column list before ENGINE and fix DEPENDS ON multi-table in CREATE MATERIALIZED VIEW (#270) 1. Column list before ENGINE (new): ClickHouse SHOW CREATE TABLE for refreshable materialized views with ENGINE (e.g. Memory) outpu... push web-flow github
51.66
25711855554 master Allow reserved keywords as bare column names in SELECT list (#269) Extend keyword-as-identifier handling to recognize end-of-statement (EOF or `;`) as a disambiguator in expression position, so projections like `SELECT case`, `SELECT case;`, or `... push web-flow github
51.62
25662343573 master Fix should allow trailing comma in SELECT list before clause keyword (#268) ClickHouse accepts `SELECT a, FROM t` (e.g. when a column is commented out with `--`). Break the SELECT-item loop after consuming a comma if the next token is a clause-st... push web-flow github
51.47
25646468838 master Add support of REGEXP as binary operator (#267) push web-flow github
51.46
24923153228 master fix: ensure projections can have either an order by or a group by (#266) push web-flow github
51.46
24572473754 master Add COMMENT support to CREATE VIEW parser (#265) CREATE VIEW in ClickHouse supports COMMENT between the schema and AS SELECT: CREATE VIEW db.v (columns) COMMENT '{...}' AS SELECT ... The parser previously failed with 'expected EOF or ; but go... push web-flow github
51.45
24495309140 master Add support for CREATE USER VALID UNTIL clause (#264) push web-flow github
51.45
24169250933 master Support COMMENT before AS SELECT in CREATE MATERIALIZED VIEW (#262) ClickHouse 26.2+ changed SHOW CREATE TABLE output to place COMMENT before AS SELECT, while 25.x places it after. The parser only supported COMMENT after AS SELECT, causing parse ... push web-flow github
51.44
23675594783 master fix: Panic on invalid number as query (#260) Co-authored-by: Kemal Hadimli <disq@users.noreply.github.com> push web-flow github
51.43
23238003604 master Add support for window names in WindowExpr (#259) push web-flow github
51.44
See All Builds (468)

Badge your Repo: clickhouse-sql-parser

We detected this repo isn’t badged! Grab the embed code to the right, add it to your repo to show off your code coverage, and when the badge is live hit the refresh button to remove this message.
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

Refresh