[TC-270] - cache_stats - query from summarized data (continuous queries) instead of raw data #545
Conversation
mitchell852
force-pushed
the
tc-270-cache-stats-cq
branch
from
8edaa87
to
072a877
Compare
| GROUP BY time($args->{interval}), cdn" | ||
| ); | ||
| my $query = qq[SELECT mean(value), percentile(value, 5), percentile(value, 95), percentile(value, 98), min(value), max(value), sum(value), count(value) | ||
| FROM "monthly"."$args->{series_name}.cdn.1min" |
There was a problem hiding this comment.
we probably want a way to change the retention policy here. Either through configs or args. hard-coding to "monthly" could lead to issues.
There was a problem hiding this comment.
The RT name is also hard-coded in the deliveryservice_stats endpoint - https://github.com/apache/incubator-trafficcontrol/blob/master/traffic_ops/app/lib/Extensions/TrafficStats/Builder/DeliveryServiceStatsBuilder.pm#L81
How about I create a new Jira and a subsequent PR that removes hardcoded RT references from both cache_stats and deliveryservice_stats endpoints and maybe puts the RT names in this file: https://github.com/apache/incubator-trafficcontrol/blob/master/traffic_ops/app/conf/production/influxdb.conf
| cdn ORDER BY asc" | ||
| ); | ||
| my $query = qq[SELECT sum(value)/count(value) | ||
| FROM "monthly"."$args->{series_name}.cdn.1min" |
There was a problem hiding this comment.
same here, we probably want a way to config the retention policy to use.
No description provided.