search

juju / charmstore

The charm store server.
http://gopkg.in/juju/charmstore.v5
GNU Affero General Public License v3.0
15 stars 37 forks source link

reorder stats key #856

Closed rogpeppe closed 5 years ago

rogpeppe commented 5 years ago

This means that we can summarize over all series for a charm in a single request.

We also change the Counters logic to add a fast path for non-list stats queries.

We also avoid adding ArchiveUploadFailed entries to the stats as they're the most common entry and we don't need the noise.

Also change the stats granularity to one hour so we get less records overall.

Note that this is targeted at the "faster-stats" feature branch because it can't be used until the relevant migration has been done.

jujugui commented 5 years ago

Refer to this link for build results (access rights to CI server needed): http://ci-gce.jujugui.org:8080//job/charmstore/2289/

jujugui commented 5 years ago

Refer to this link for build results (access rights to CI server needed): http://ci-gce.jujugui.org:8080//job/charmstore/2290/

jujugui commented 5 years ago

Refer to this link for build results (access rights to CI server needed): http://ci-gce.jujugui.org:8080//job/charmstore/2291/

jujugui commented 5 years ago

Refer to this link for build results (access rights to CI server needed): http://ci-gce.jujugui.org:8080//job/charmstore/2292/

rogpeppe commented 5 years ago

:shipit: $$merge$$

jujugui commented 5 years ago

Status: merge request accepted. Url: http://ci-gce.jujugui.org:8080/job/charmstore-merge

jujugui commented 5 years ago

Refer to this link for build results (access rights to CI server needed): http://ci-gce.jujugui.org:8080//job/charmstore/2293/