Differential D3922

cli: speedup the `swh` cli command startup time
ClosedPublic
Actions

Authored by douardda on Sep 10 2020, 4:27 PM.

Details

Reviewers

vlorentz

Group Reviewers

Reviewers

Maniphest Tasks

T2575: Investigate if/how we could improve `swh` cli command startup time

Commits

rDVAUc0c3a897420d: cli: speedup the `swh` cli command startup time

Summary

by moving import statements in functions. Also simplify the main
help message to to get rid of the necessity to import swh.core.config.

Related to T2575.

Diff Detail

Repository

rDVAU Software Heritage Vault

Lint

Automatic diff as part of commit; lint not applicable.

Unit

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

Event Timeline

douardda created this revision.Sep 10 2020, 4:27 PM

Herald added a reviewer: Reviewers. · View Herald TranscriptSep 10 2020, 4:27 PM

Build is green

Patch application report for D3922 (id=13782)

Rebasing onto 5cc96ef49e...

Current branch diff-target is up to date.

Changes applied before test

commit c0c3a897420ddd47db735ed5b05fccaf59202b36
Author: David Douard <david.douard@sdfa3.org>
Date:   Wed Sep 9 19:05:03 2020 +0200

    cli: speedup the `swh` cli command startup time
    
    by moving import statements in functions. Also simplify the main
    help message to to get rid of the necessity to import swh.core.config.
    
    Related to T2575.

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

Harbormaster completed remote builds in B15009: Diff 13782.Sep 10 2020, 4:28 PM

vlorentz accepted this revision.Sep 10 2020, 4:45 PM

This revision is now accepted and ready to land.Sep 10 2020, 4:45 PM

Closed by commit rDVAUc0c3a897420d: cli: speedup the `swh` cli command startup time (authored by douardda). · Explain WhySep 10 2020, 5:12 PM

This revision was automatically updated to reflect the committed changes.

douardda added a commit: rDVAUc0c3a897420d: cli: speedup the `swh` cli command startup time.