-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Add aggregate_last_24_hours option #11157
Conversation
This reverts commit 8d5b0c3.
@@ -141,6 +141,14 @@ files: | |||
- snowflake.billing | |||
- snowflake.storage | |||
- snowflake.logins | |||
- name: use_deprecated_aggregation |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what do you think about making the option more descriptive, something like aggregate_24_hours
? There could be instances where someone would prefer aggregating on the current day
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was hesitant to use something like aggregate_24_hours
; as aggregate_24_hours: false
does not give that much info on how the queries are aggregated. I'll change the name of the option though
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
would aggregate_last_24_hours make make more sense?
Co-authored-by: Sarah Witt <sarah.witt@datadoghq.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
What does this PR do?
The queries were aggregating entries
since the beginning of the day
, this PR add a config flag to aggregate the entries inthe last 24 hours
instead.Motivation
Since some tables could have a 3 hours latency, the queries
since the beginning of the day
could be empty and leads to missing metrics. Aggregating the entries in the last 24 hours instead is more consistent.Additional Notes