Skip to content

Enable WHERE name optimization for all queries, remove feature flag#6154

Merged
ukutaht merged 1 commit intomasterfrom
release-name-optimization
Mar 11, 2026
Merged

Enable WHERE name optimization for all queries, remove feature flag#6154
ukutaht merged 1 commit intomasterfrom
release-name-optimization

Conversation

@ukutaht
Copy link
Contributor

@ukutaht ukutaht commented Mar 11, 2026

Follow-up to #6145

Confirmed the optimization is working in production with spot checks.

@ukutaht
Copy link
Contributor Author

ukutaht commented Mar 11, 2026

Results from one test. visit:source breakdown with event:props filter from a large client. Two identical queries with the only difference being the presence of WHERE name != 'engagement' filter:

before: 65.25 GiB read bytes, query duration 3,423ms (SelectedMarks':10936, 'FunctionExecute':144,842)
after: 35.20 GiB read bytes, query duration 1,383ms ('SelectedMarks':5919, 'FunctionExecute':84,162)

This optimization will only affect larger sites (due to granule size, smaller sites will not have sufficient clustering in storage) but the improvements should be pretty broad across many queries that scan the events table.

@ukutaht ukutaht requested a review from a team March 11, 2026 12:20
Copy link
Member

@aerosol aerosol left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀

@ukutaht ukutaht added this pull request to the merge queue Mar 11, 2026
Merged via the queue into master with commit 011e687 Mar 11, 2026
23 checks passed
@ukutaht ukutaht deleted the release-name-optimization branch March 11, 2026 13:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants