Differential D3762

common/origin_visits: Optimize get_origin_visit performance
ClosedPublic
Actions

Authored by anlambert on Aug 11 2020, 7:05 PM.

Details

Reviewers

vlorentz

Group Reviewers

Reviewers

Commits

rDWAPPS0cb8a778600e: common/origin_visits: Optimize get_origin_visit performance

Summary

If the latest origin visit matches some search criteria, return it instead of
fetching the whole visits list and searching in it.

Diff Detail

Repository

rDWAPPS Web applications

Branch

optmimize-get-origin-visit

Lint

No Linters Available

Unit

No Unit Test Coverage

Build Status

Buildable 14499
Build 22304: Phabricator diff pipeline on jenkins	Jenkins console · Jenkins
Build 22303: arc lint + arc unit

Event Timeline

anlambert created this revision.Aug 11 2020, 7:05 PM

Herald added a reviewer: Reviewers. · View Herald TranscriptAug 11 2020, 7:05 PM

Build is green

Patch application report for D3762 (id=13235)

Rebasing onto 9924a39bc5...

Current branch diff-target is up to date.

Changes applied before test

commit 0cb8a778600e191fe935a3cb642772144b0d9839
Author: Antoine Lambert <antoine.lambert@inria.fr>
Date:   Tue Aug 11 19:02:51 2020 +0200

    common/origin_visits: Optimize get_origin_visit performance
    
    If the latest origin visit matches some search criteria, return it instead of
    fetching the whole visits list and searching in it.

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

Harbormaster completed remote builds in B14499: Diff 13235.Aug 11 2020, 7:12 PM

vlorentz accepted this revision.Aug 11 2020, 8:38 PM

This revision is now accepted and ready to land.Aug 11 2020, 8:38 PM

Closed by commit rDWAPPS0cb8a778600e: common/origin_visits: Optimize get_origin_visit performance (authored by anlambert). · Explain WhyAug 12 2020, 10:36 AM

This revision was automatically updated to reflect the committed changes.

anlambert added a commit: rDWAPPS0cb8a778600e: common/origin_visits: Optimize get_origin_visit performance.