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

Use ORDER BY tuple() by default in ClickHouse MergeTree #23049

Merged
merged 2 commits into from
Aug 16, 2024

Conversation

ebyhr
Copy link
Member

@ebyhr ebyhr commented Aug 15, 2024

Description

ClickHouse allows ORDER BY tuple() expression when no sorting is required with MergeTree engine.

Fixes #23048

Release notes

# ClickHouse
* Add support for creating tables with `MergeTree` engine without `order_by` table property. ({issue}`23048`)

@cla-bot cla-bot bot added the cla-signed label Aug 15, 2024
@ebyhr ebyhr changed the title Ebi/clickhouse margetree tuple Use ORDER BY tuple() by default in ClickHouse MergeTree Aug 15, 2024
@ebyhr ebyhr force-pushed the ebi/clickhouse-margetree-tuple branch from 30737c2 to 86290e2 Compare August 15, 2024 05:22
@github-actions github-actions bot added the docs label Aug 15, 2024
@ebyhr ebyhr force-pushed the ebi/clickhouse-margetree-tuple branch 2 times, most recently from 23a89dd to 49422c5 Compare August 15, 2024 06:04
@ebyhr ebyhr requested review from wendigo and hashhar August 15, 2024 22:40
@ebyhr ebyhr force-pushed the ebi/clickhouse-margetree-tuple branch from 49422c5 to b0caaae Compare August 16, 2024 00:26
@ebyhr ebyhr merged commit ca19c7f into trinodb:master Aug 16, 2024
5 of 13 checks passed
@ebyhr ebyhr deleted the ebi/clickhouse-margetree-tuple branch August 16, 2024 00:27
@github-actions github-actions bot added this to the 455 milestone Aug 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

Incorrect parsing ClickHouse query
2 participants