Differential D5502

Add a method get several counter values at once
ClosedPublic
Actions

Authored by vsellier on Apr 13 2021, 5:43 PM.

Details

Reviewers

ardumont

Group Reviewers

Reviewers

Maniphest Tasks

T3231: Make the source of the object counts configurable

Commits

rDCNT38eab384f275: Add a method get several counter values at once

Summary

It allow the webapp to retrieve all the values it needs for the
homepage in only one rpc call

Related to T3231

Diff Detail

Repository

rDCNT Archive counters

Lint

Automatic diff as part of commit; lint not applicable.

Unit

Automatic diff as part of commit; unit tests not applicable.

Event Timeline

vsellier created this revision.Apr 13 2021, 5:43 PM

Herald added a reviewer: Reviewers. · View Herald TranscriptApr 13 2021, 5:43 PM

Build is green

Patch application report for D5502 (id=19652)

Rebasing onto 46d1d61bb9...

Current branch diff-target is up to date.

Changes applied before test

commit aea21fbdb8e02ff6db51d066c36e9e52cefa3cfb
Author: Vincent SELLIER <vincent.sellier@softwareheritage.org>
Date:   Tue Apr 13 17:40:04 2021 +0200

    Add a method get several counter values at once
    
    It allow the webapp to retrieve all the values it needs for the
    homepage in only one rpc call
    
    Related to T3232

See https://jenkins.softwareheritage.org/job/DCNT/job/tests-on-diff/37/ for more details.

Harbormaster completed remote builds in B20680: Diff 19652.Apr 13 2021, 5:44 PM

vsellier requested review of this revision.Apr 13 2021, 5:44 PM

ardumont accepted this revision.Apr 13 2021, 5:59 PM

This revision is now accepted and ready to land.Apr 13 2021, 5:59 PM

Reference the right task

vsellier edited the summary of this revision. (Show Details)Apr 13 2021, 6:01 PM

vsellier added a task: T3231: Make the source of the object counts configurable.

vsellier removed a task: T3232: remove hardcoded historical values from the webapp.

Build is green

Patch application report for D5502 (id=19654)

Rebasing onto 46d1d61bb9...

Current branch diff-target is up to date.

Changes applied before test

commit 38eab384f275fa3e2e78de0d26530149d162d32c
Author: Vincent SELLIER <vincent.sellier@softwareheritage.org>
Date:   Tue Apr 13 17:40:04 2021 +0200

    Add a method get several counter values at once
    
    It allow the webapp to retrieve all the values it needs for the
    homepage in only one rpc call
    
    Related to T3231

See https://jenkins.softwareheritage.org/job/DCNT/job/tests-on-diff/38/ for more details.

Harbormaster completed remote builds in B20681: Diff 19654.Apr 13 2021, 6:02 PM

Closed by commit rDCNT38eab384f275: Add a method get several counter values at once (authored by vsellier). · Explain WhyApr 13 2021, 7:09 PM

This revision was automatically updated to reflect the committed changes.

vsellier added a commit: rDCNT38eab384f275: Add a method get several counter values at once.