Clickhouse allow_deprecated_syntax_for_merge_tree

Open mdsohelmia opened this issue 2 years ago • 2 comments

Clickhouse Version 23.5.2.7

2023/06/13 11:13:24 migrate run: failed to ensure DB version: code: 36, message: This syntax for *MergeTree engine is deprecated. Use extended storage definition syntax with ORDER BY/PRIMARY KEY clause. See also allow_deprecated_syntax_for_merge_tree setting. exit status 1

Jun 13 '23 11:06 mdsohelmia

Relevant bit,

https://github.com/pressly/goose/blob/master/internal/dialect/dialectquery/clickhouse.go#L16-L17

although what does it mean for existing clickhouse instances that were created with the "old" query?

There's a few PR's from @iamluc @arunk-s related to clickhouse, so we should probably bump those and see if they also fix this issue:

https://github.com/pressly/goose/pull/520 https://github.com/pressly/goose/pull/530

Jun 13 '23 13:06 mfridman