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

Clickhouse allow_deprecated_syntax_for_merge_tree #535

Open
mdsohelmia opened this issue Jun 13, 2023 · 1 comment
Open

Clickhouse allow_deprecated_syntax_for_merge_tree #535

mdsohelmia opened this issue Jun 13, 2023 · 1 comment

Comments

@mdsohelmia
Copy link

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

@mfridman
Copy link
Collaborator

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:

#520
#530

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

No branches or pull requests

2 participants