goose icon indicating copy to clipboard operation
goose copied to clipboard

Set allow_deprecated_syntax_for_merge_tree in ClickHouse tests

Open mfridman opened this issue 3 years ago • 0 comments

There was a breaking change in ClickHouse version 22.7. More details here. https://github.com/ClickHouse/ClickHouse/pull/38335

Ordinary database engine and old storage definition syntax for *MergeTree tables are deprecated. By default it's not possible to create new databases with Ordinary engine. If system database has Ordinary engine it will be automatically converted to Atomic on server startup. There are settings to keep old behavior (allow_deprecated_database_ordinary and allow_deprecated_syntax_for_merge_tree), but these settings may be removed in future releases. #38335 (Alexander Tokmakov).

In goose we create a new database with the following syntax, note the Engine = MergeTree.

CREATE TABLE IF NOT EXISTS goose_db_version (
    version_id Int64,
    is_applied UInt8,
    date Date default now(),
    tstamp DateTime default now()
  ) Engine = MergeTree(date, (date), 8192);

For now, let's set allow_deprecated_syntax_for_merge_tree to true in our tests.

mfridman avatar Jul 30 '22 13:07 mfridman