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

fix(streaming): alias meter query where #1607

Merged
merged 2 commits into from
Oct 2, 2024
Merged

Conversation

hekike
Copy link
Contributor

@hekike hekike commented Oct 2, 2024

Alias meter table and prefix columns are used in the where condition to avoid shadowing by aggregated columns.
This fixes querying hourly and daily windows as well as filters the underlying per-minute windows, not the rounded tumbling window.

@hekike hekike requested a review from tothandras October 2, 2024 01:09
@hekike hekike added area/processor release-note/bug-fix Release note: Bug Fixes labels Oct 2, 2024
@hekike hekike requested a review from tothandras October 2, 2024 13:57
@hekike hekike merged commit b4a5836 into main Oct 2, 2024
18 checks passed
@hekike hekike deleted the fix/meter-query-alias-where branch October 2, 2024 13:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants