common/identifiers: Remove branch name processing in SWHID resolver
103b85d1c6c2
Actions

Description

common/identifiers: Remove branch name processing in SWHID resolver

Previously the SWHID resolver was trying to associate a branch name to
a revision by checking tip revisions in snapshot branches list.

That operation is not really needed, can be costly to execute and can
lead to regressions in SWHID resolving so remove it.

Related to T3195

Details

Provenance

anlambert	Authored on Apr 1 2021, 5:31 PM
anlambert	Pushed on Apr 1 2021, 6:21 PM

Differential Revision

D5404: common/identifiers: Remove branch name processing in SWHID resolver

Parents

rDWAPPS07f06cba53b1: swh-web: Migrate from sqlite to postgresql

Branches

Unknown

Tags

Unknown

Tasks

T3195: Some SWHIDs provided by the WebApp produce 404 when resolved

Build Status

Buildable 20403
Build 31664: test-and-build	Jenkins console · Jenkins

Changes (4)

Path

Size

swh/

web/

common/

archive.py

identifiers.py

tests/

common/

test_archive.py

test_identifiers.py

rDWAPPS103b85d1c6c2

View Options

swh/web/common/archive.py

View Options

swh/web/common/identifiers.py

View Options

swh/web/tests/common/test_archive.py

View Options

swh/web/tests/common/test_identifiers.py

common/identifiers: Remove branch name processing in SWHID resolver103b85d1c6c2Actions

Description

Details

Event Timeline

Changes (4)

rDWAPPS103b85d1c6c2

swh/web/common/archive.py

swh/web/common/identifiers.py

swh/web/tests/common/test_archive.py

swh/web/tests/common/test_identifiers.py

common/identifiers: Remove branch name processing in SWHID resolver
103b85d1c6c2
Actions