Page MenuHomeSoftware Heritage

Add `algos.identifier.identifiers_missing`
Changes PlannedPublicDraft

Authored by Ericson2314 on May 5 2022, 8:13 PM.

Details

Reviewers
None
Group Reviewers
Reviewers
Summary

This is a small wrapper around the underlying *_missing functions in
order to assist with looking up missing SWHIDs.

This will be a building block to clean up some things in swh-web so that
https://forge.softwareheritage.org/T4224 doesn't come back.

Diff Detail

Repository
rDSTO Storage manager
Branch
identifiers-missing (branched from master)
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 29102
Build 45499: Phabricator diff pipeline on jenkinsJenkins console · Jenkins
Build 45498: arc lint + arc unit

Event Timeline

Build has FAILED

Patch application report for D7751 (id=28025)

Rebasing onto 9562953493...

Current branch diff-target is up to date.
Changes applied before test
commit 31d4c3e4cfee780a1e3c823ef132e216705d8704
Author: John Ericson <John.Ericson@Obsidian.Systems>
Date:   Thu May 5 13:25:57 2022 -0400

    Add `algos.identifier.identifiers_missing`
    
    This is a small wrapper around the underlying `*_missing` functions in
    order to assist with looking up missing SWHIDs.
    
    This will be a building block to clean up some things in swh-web so that
    https://forge.softwareheritage.org/T4224 doesn't come back.

Link to build: https://jenkins.softwareheritage.org/job/DSTO/job/tests-on-diff/1605/
See console output for more information: https://jenkins.softwareheritage.org/job/DSTO/job/tests-on-diff/1605/console

Build has FAILED

Patch application report for D7751 (id=28026)

Rebasing onto 9562953493...

Current branch diff-target is up to date.
Changes applied before test
commit 026af46d2e36b5c40309ad1a1c13549167c2ff2e
Author: John Ericson <John.Ericson@Obsidian.Systems>
Date:   Thu May 5 13:25:57 2022 -0400

    Add `algos.identifier.identifiers_missing`
    
    Summary:
    This is a small wrapper around the underlying `*_missing` functions in
    order to assist with looking up missing SWHIDs.
    
    This will be a building block to clean up some things in swh-web so that
    https://forge.softwareheritage.org/T4224 doesn't come back.
    
    Reviewers: #reviewers
    
    Differential Revision: https://forge.softwareheritage.org/D7751

Link to build: https://jenkins.softwareheritage.org/job/DSTO/job/tests-on-diff/1606/
See console output for more information: https://jenkins.softwareheritage.org/job/DSTO/job/tests-on-diff/1606/console

Harbormaster returned this revision to the author for changes because remote builds failed.May 5 2022, 8:16 PM
Harbormaster failed remote builds in B29102: Diff 28026!

Ensure thre is an error on invalid object type

Condense / deduplicate to the code too in the process.

Build has FAILED

Patch application report for D7751 (id=28027)

Rebasing onto 9562953493...

Current branch diff-target is up to date.
Changes applied before test
commit a92cef818b5a2b947f2cf733283c3f28116a2797
Author: John Ericson <John.Ericson@Obsidian.Systems>
Date:   Thu May 5 13:25:57 2022 -0400

    Add `algos.identifier.identifiers_missing`
    
    Summary:
    This is a small wrapper around the underlying `*_missing` functions in
    order to assist with looking up missing SWHIDs.
    
    This will be a building block to clean up some things in swh-web so that
    https://forge.softwareheritage.org/T4224 doesn't come back.
    
    Reviewers: #reviewers
    
    Differential Revision: https://forge.softwareheritage.org/D7751

Link to build: https://jenkins.softwareheritage.org/job/DSTO/job/tests-on-diff/1607/
See console output for more information: https://jenkins.softwareheritage.org/job/DSTO/job/tests-on-diff/1607/console

Harbormaster returned this revision to the author for changes because remote builds failed.May 5 2022, 8:30 PM
Harbormaster failed remote builds in B29103: Diff 28027!

Need to drain iterable to set

Build was aborted

Patch application report for D7751 (id=28028)

Rebasing onto 9562953493...

Current branch diff-target is up to date.
Changes applied before test
commit 1717c3eaa70a5c8e4c73dd6ac59926e8dd9f3f8e
Author: John Ericson <John.Ericson@Obsidian.Systems>
Date:   Thu May 5 13:25:57 2022 -0400

    Add `algos.identifier.identifiers_missing`
    
    Summary:
    This is a small wrapper around the underlying `*_missing` functions in
    order to assist with looking up missing SWHIDs.
    
    This will be a building block to clean up some things in swh-web so that
    https://forge.softwareheritage.org/T4224 doesn't come back.
    
    Reviewers: #reviewers
    
    Differential Revision: https://forge.softwareheritage.org/D7751

Link to build: https://jenkins.softwareheritage.org/job/DSTO/job/tests-on-diff/1608/
See console output for more information: https://jenkins.softwareheritage.org/job/DSTO/job/tests-on-diff/1608/console

Harbormaster returned this revision to the author for changes because remote builds failed.May 5 2022, 8:54 PM
Harbormaster failed remote builds in B29104: Diff 28028!