Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Set allow_deprecated_syntax_for_merge_tree in ClickHouse tests #392

Closed
mfridman opened this issue Jul 30, 2022 · 0 comments · Fixed by #408
Closed

Set allow_deprecated_syntax_for_merge_tree in ClickHouse tests #392

mfridman opened this issue Jul 30, 2022 · 0 comments · Fixed by #408

Comments

@mfridman
Copy link
Collaborator

There was a breaking change in ClickHouse version 22.7. More details here. ClickHouse/ClickHouse#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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant