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 "mix" shuffle spec for target size with nil clusterBy. #17810

Merged
merged 1 commit into from
Mar 19, 2025

Conversation

gianm
Copy link
Contributor

@gianm gianm commented Mar 17, 2025

When a nil clusterBy is used, we have no way of achieving a particular target size, so we need to fall back to a "mix" spec (unsorted single partition).

This comes up for queries like SELECT COUNT(*) FROM FOO LIMIT 1 when results use a target size, such as when we are inserting into another table or when we are writing to durable storage.

When a nil clusterBy is used, we have no way of achieving a particular
target size, so we need to fall back to a "mix" spec (unsorted single
partition).

This comes up for queries like "SELECT COUNT(*) FROM FOO LIMIT 1" when
results use a target size, such as when we are inserting into another
table or when we are writing to durable storage.
@gianm gianm added Bug Area - MSQ For multi stage queries - https://github.com/apache/druid/issues/12262 labels Mar 17, 2025
@cryptoe cryptoe requested a review from adarshsanjeev March 18, 2025 08:28
@cryptoe cryptoe merged commit 2300bb4 into apache:master Mar 19, 2025
41 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area - Batch Ingestion Area - MSQ For multi stage queries - https://github.com/apache/druid/issues/12262 Bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants