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
Forbid SimpleAggregateFunction in ORDER BY of MergeTree tables #61399
Forbid SimpleAggregateFunction in ORDER BY of MergeTree tables #61399
Conversation
This is an automated comment for commit 8936528 with description of existing statuses. It's updated for the latest CI running ❌ Click here to open a full report in a separate page
Successful checks
|
d28b930
to
9bf34c3
Compare
Thank you, this is a good change! |
47dd684
to
89eaa3c
Compare
89eaa3c
to
8b77f4e
Compare
Will try a rebase. UPD: though it looks like it fails in lots of PRs |
Like AggregateFunction is forbidden, but they are forbidden because they are not comparable. New setting (allow_suspicious_primary_key) had been added for backard compatiblity (turned OFF by default). Signed-off-by: Azat Khuzhin <a.khuzhin@semrush.com>
8b77f4e
to
11d0b0a
Compare
@azat, why don't just fix it? Otherwise, it is wasting resources. |
Changelog category (leave one):
Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Forbid
SimpleAggregateFunction
inORDER BY
ofMergeTree
tables (likeAggregateFunction
is forbidden, but they are forbidden because they are not comparable) by default (useallow_suspicious_primary_key
to allow them).