Page MenuHomeSoftware Heritage
Differential D5081

Add new class QualifiedSWHID to replace SWHID, and deprecate the latter.
ClosedPublic
Actions

Authored by vlorentz on Feb 16 2021, 1:34 PM.

Details

Reviewers
ardumont
zack
Group Reviewers
Reviewers
Commits
rDMOD690b7f824f55: Add new class QualifiedSWHID to replace SWHID, and deprecate the latter.
Summary

Following the discussion on T3034, we decided to replace SWHID with
two or three classes:

  • QualifiedSWHID to replace the existing SWHID (standard types + qualifiers)
  • CoreSWHID, for "core SWHID" only (standard types + no qualifiers)
  • ExtendedSWHID for internal use in Software Heritage (extra types + no qualifiers)

Since migrating from SWHID will break existing code, this commit uses
the opportunity to modernize it a little, ie.:

  • keyword-only constructor, to get rid of the hacky default values for object_type and object_id
  • enum instead of strings for the object type
  • bytes instead of an hex string for the object id
  • rename metadata to qualifiers

Diff Detail

Repository
rDMOD Data model
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

There are a very large number of changes, so older changes are hidden. Show Older Changes
swh-public-ci added a comment.Feb 16 2021, 1:37 PM

Build is green

Patch application report for D5081 (id=18141)

Rebasing onto 0c16581283...

Current branch diff-target is up to date.
Changes applied before test
commit 6af9d198ad3a4a44681b5f66d801a4ef53b3a1f0
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Tue Feb 16 13:33:13 2021 +0100

    Add new class QualifiedSWHID to replace SWHID, and deprecate the latter.
    
    Following the discussion on T3034, we decided to replace SWHID with
    two or three classes:
    
    * QualifiedSWHID to replace the existing SWHID (standard types + qualifiers)
    * ExtendedSWHID for internal use in Software Heritage (extra types + no qualifiers)
    * maybe CoreSWHID, for "core SWHID" only (standard types + qualifiers)

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

Harbormaster completed remote builds in B19245: Diff 18141.Feb 16 2021, 1:37 PM
vlorentz updated this revision to Diff 18142.Feb 16 2021, 1:37 PM

better commit msg

vlorentz edited the summary of this revision. (Show Details)Feb 16 2021, 1:38 PM
vlorentz edited the summary of this revision. (Show Details)
swh-public-ci added a comment.Feb 16 2021, 1:39 PM

Build is green

Patch application report for D5081 (id=18142)

Rebasing onto 0c16581283...

Current branch diff-target is up to date.
Changes applied before test
commit 166a856916aefd99ace8ebb934b821e05a0f6b43
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Tue Feb 16 13:33:13 2021 +0100

    Add new class QualifiedSWHID to replace SWHID, and deprecate the latter.
    
    Following the discussion on T3034, we decided to replace SWHID with
    two or three classes:
    
    * QualifiedSWHID to replace the existing SWHID (standard types + qualifiers)
    * ExtendedSWHID for internal use in Software Heritage (extra types + no qualifiers)
    * maybe CoreSWHID, for "core SWHID" only (standard types + qualifiers)
    
    Since migrating from SWHID will break existing code, this commit uses
    the opportunity to modernize it a little, ie.:
    
    * `keyword`-only constructor, to get rid of the hacky default values for
      `object_type` and `object_id`
    * enum instead of strings for the object type
    * `bytes` instead of an hex string for the object id
    * rename `metadata` to `qualifiers`

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

Harbormaster completed remote builds in B19246: Diff 18142.Feb 16 2021, 1:39 PM
zack requested changes to this revision.Feb 16 2021, 2:26 PM
zack added a subscriber: zack.
zack added inline comments.
swh/model/identifiers.py
25

Thanks gawd yes.

  • minor (1): can you add a docstring for this enum, even a oneliner would do
  • minor (2): I think it's more customary to from enum import Enum
740–751

can you separate attributes with empty lines here?
without them it is hard to visually associate docstrings to the right field

753–754

I've a grudge with a series of separate validators here, considering most of the syntax (except qualifiers) can be easily validated with a regex, which has also been shown in the past to be faster (cf.: 574685052348bfc6ed28570f06b9cc4302dfde27). Can we use a single regex and validator instead (for core unqualified SWHIDs)?

771

leftover print?

798–800

This is quadratic on the number of qualifiers.
Can we use something-str.join-something instead?

830

should we log a deprecation message here? or is it too soon to do that?

This revision now requires changes to proceed.Feb 16 2021, 2:26 PM
vlorentz added inline comments.Feb 16 2021, 2:34 PM
swh/model/identifiers.py
753–754

No, because using a single regex is only possible when parsing a string. Validators are also used when building the object from parts.

798–800

I copy-pasted from the existing code, so it's out of scope for this diff.

830

oops, it got lost before I committed.

vlorentz updated this revision to Diff 18144.Feb 16 2021, 2:41 PM
vlorentz edited the summary of this revision. (Show Details)

apply all of @zack's comments

swh-public-ci added a comment.Feb 16 2021, 2:43 PM

Build has FAILED

Patch application report for D5081 (id=18144)

Rebasing onto 0c16581283...

Current branch diff-target is up to date.
Changes applied before test
commit 62bcc5fb93c424b2b013347595e958fb93b5ada5
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Tue Feb 16 13:33:13 2021 +0100

    Add new class QualifiedSWHID to replace SWHID, and deprecate the latter.
    
    Following the discussion on T3034, we decided to replace SWHID with
    two or three classes:
    
    * QualifiedSWHID to replace the existing SWHID (standard types + qualifiers)
    * ExtendedSWHID for internal use in Software Heritage (extra types + no qualifiers)
    * maybe CoreSWHID, for "core SWHID" only (standard types + qualifiers)
    
    Since migrating from SWHID will break existing code, this commit uses
    the opportunity to modernize it a little, ie.:
    
    * `keyword`-only constructor, to get rid of the hacky default values for
      `object_type` and `object_id`
    * enum instead of strings for the object type
    * `bytes` instead of an hex string for the object id
    * rename `metadata` to `qualifiers`

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

Harbormaster failed remote builds in B19248: Diff 18144!Feb 16 2021, 2:43 PM
vlorentz updated this revision to Diff 18145.Feb 16 2021, 2:46 PM

rm dbg traceback

swh-public-ci added a comment.Feb 16 2021, 2:47 PM

Build is green

Patch application report for D5081 (id=18145)

Rebasing onto 0c16581283...

Current branch diff-target is up to date.
Changes applied before test
commit 7505edc8e026713fafec933475d300b097fdbf5a
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Tue Feb 16 13:33:13 2021 +0100

    Add new class QualifiedSWHID to replace SWHID, and deprecate the latter.
    
    Following the discussion on T3034, we decided to replace SWHID with
    two or three classes:
    
    * QualifiedSWHID to replace the existing SWHID (standard types + qualifiers)
    * ExtendedSWHID for internal use in Software Heritage (extra types + no qualifiers)
    * maybe CoreSWHID, for "core SWHID" only (standard types + qualifiers)
    
    Since migrating from SWHID will break existing code, this commit uses
    the opportunity to modernize it a little, ie.:
    
    * `keyword`-only constructor, to get rid of the hacky default values for
      `object_type` and `object_id`
    * enum instead of strings for the object type
    * `bytes` instead of an hex string for the object id
    * rename `metadata` to `qualifiers`

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

Harbormaster completed remote builds in B19249: Diff 18145.Feb 16 2021, 2:47 PM
zack resigned from this revision.EditedFeb 16 2021, 3:07 PM

LGTM.

I'm not approving it (and only resigning as reviewer instead) just to allow others to have a look, given it's something quite foundational in the data model.

ardumont added a subscriber: ardumont.Feb 16 2021, 7:17 PM

Did not read the tests yet.

swh/model/identifiers.py
784

Missing scenarios it seems.

809

from where does _object_type_map come from?

(and missing test)

douardda added a subscriber: douardda.Feb 17 2021, 2:07 PM

(as mentioned on IRC): Since we are doing big works, I see no reason to keep the 'qualifiers' a schema-less dict-like attribute. I'd rather have them as attributes.

vlorentz added inline comments.Feb 18 2021, 12:10 PM
swh/model/identifiers.py
809

it's one of the many constants at the toplevel, I'll clean it up when removing SWHID

vlorentz updated this revision to Diff 18232.Feb 18 2021, 12:29 PM

copy missing tests

swh-public-ci added a comment.Feb 18 2021, 12:31 PM

Build is green

Patch application report for D5081 (id=18232)

Rebasing onto 0c16581283...

Current branch diff-target is up to date.
Changes applied before test
commit dbd8b903f1789e7c15445966e9577fc5f3d3a9bb
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Tue Feb 16 13:33:13 2021 +0100

    Add new class QualifiedSWHID to replace SWHID, and deprecate the latter.
    
    Following the discussion on T3034, we decided to replace SWHID with
    two or three classes:
    
    * QualifiedSWHID to replace the existing SWHID (standard types + qualifiers)
    * ExtendedSWHID for internal use in Software Heritage (extra types + no qualifiers)
    * maybe CoreSWHID, for "core SWHID" only (standard types + qualifiers)
    
    Since migrating from SWHID will break existing code, this commit uses
    the opportunity to modernize it a little, ie.:
    
    * `keyword`-only constructor, to get rid of the hacky default values for
      `object_type` and `object_id`
    * enum instead of strings for the object type
    * `bytes` instead of an hex string for the object id
    * rename `metadata` to `qualifiers`

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

Harbormaster completed remote builds in B19329: Diff 18232.Feb 18 2021, 12:31 PM
vlorentz updated this revision to Diff 18234.Feb 18 2021, 12:41 PM

add serialization tests (for both SWHID and QualifiedSWHID)

vlorentz marked 3 inline comments as done.Feb 18 2021, 12:42 PM
swh-public-ci added a comment.Feb 18 2021, 12:42 PM

Build is green

Patch application report for D5081 (id=18234)

Rebasing onto 0c16581283...

Current branch diff-target is up to date.
Changes applied before test
commit ad5c39e895c83e38c04e65748b6758fb34478375
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Tue Feb 16 13:33:13 2021 +0100

    Add new class QualifiedSWHID to replace SWHID, and deprecate the latter.
    
    Following the discussion on T3034, we decided to replace SWHID with
    two or three classes:
    
    * QualifiedSWHID to replace the existing SWHID (standard types + qualifiers)
    * ExtendedSWHID for internal use in Software Heritage (extra types + no qualifiers)
    * maybe CoreSWHID, for "core SWHID" only (standard types + qualifiers)
    
    Since migrating from SWHID will break existing code, this commit uses
    the opportunity to modernize it a little, ie.:
    
    * `keyword`-only constructor, to get rid of the hacky default values for
      `object_type` and `object_id`
    * enum instead of strings for the object type
    * `bytes` instead of an hex string for the object id
    * rename `metadata` to `qualifiers`

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

Harbormaster completed remote builds in B19331: Diff 18234.Feb 18 2021, 12:42 PM
vlorentz updated this revision to Diff 18235.Feb 18 2021, 12:45 PM

100% test coverage

swh-public-ci added a comment.Feb 18 2021, 12:47 PM

Build is green

Patch application report for D5081 (id=18235)

Rebasing onto 0c16581283...

Current branch diff-target is up to date.
Changes applied before test
commit e8c334c8da5fc342e1a902c454c6b00782bd5eb6
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Tue Feb 16 13:33:13 2021 +0100

    Add new class QualifiedSWHID to replace SWHID, and deprecate the latter.
    
    Following the discussion on T3034, we decided to replace SWHID with
    two or three classes:
    
    * QualifiedSWHID to replace the existing SWHID (standard types + qualifiers)
    * ExtendedSWHID for internal use in Software Heritage (extra types + no qualifiers)
    * maybe CoreSWHID, for "core SWHID" only (standard types + qualifiers)
    
    Since migrating from SWHID will break existing code, this commit uses
    the opportunity to modernize it a little, ie.:
    
    * `keyword`-only constructor, to get rid of the hacky default values for
      `object_type` and `object_id`
    * enum instead of strings for the object type
    * `bytes` instead of an hex string for the object id
    * rename `metadata` to `qualifiers`

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

Harbormaster completed remote builds in B19332: Diff 18235.Feb 18 2021, 12:47 PM
vlorentz updated this revision to Diff 18236.Feb 18 2021, 12:53 PM

fix test

vlorentz added a comment.Feb 18 2021, 12:54 PM
In D5081#128603, @douardda wrote:

(as mentioned on IRC): Since we are doing big works, I see no reason to keep the 'qualifiers' a schema-less dict-like attribute. I'd rather have them as attributes.

I'll do this in a future diff

swh-public-ci added a comment.Feb 18 2021, 12:54 PM

Build is green

Patch application report for D5081 (id=18236)

Rebasing onto 0c16581283...

Current branch diff-target is up to date.
Changes applied before test
commit a69394b8c174bc4a80bbacf5c933ac19b1a3ebf3
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Tue Feb 16 13:33:13 2021 +0100

    Add new class QualifiedSWHID to replace SWHID, and deprecate the latter.
    
    Following the discussion on T3034, we decided to replace SWHID with
    two or three classes:
    
    * QualifiedSWHID to replace the existing SWHID (standard types + qualifiers)
    * ExtendedSWHID for internal use in Software Heritage (extra types + no qualifiers)
    * maybe CoreSWHID, for "core SWHID" only (standard types + qualifiers)
    
    Since migrating from SWHID will break existing code, this commit uses
    the opportunity to modernize it a little, ie.:
    
    * `keyword`-only constructor, to get rid of the hacky default values for
      `object_type` and `object_id`
    * enum instead of strings for the object type
    * `bytes` instead of an hex string for the object id
    * rename `metadata` to `qualifiers`

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

Harbormaster completed remote builds in B19333: Diff 18236.Feb 18 2021, 12:54 PM
vlorentz updated this revision to Diff 18237.Feb 18 2021, 1:11 PM

fix commit msg

vlorentz edited the summary of this revision. (Show Details)Feb 18 2021, 1:11 PM
swh-public-ci added a comment.Feb 18 2021, 1:12 PM

Build is green

Patch application report for D5081 (id=18237)

Rebasing onto 0c16581283...

Current branch diff-target is up to date.
Changes applied before test
commit bc91ea91b3c38b96fc461097b0d55e889bcf893a
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Tue Feb 16 13:33:13 2021 +0100

    Add new class QualifiedSWHID to replace SWHID, and deprecate the latter.
    
    Following the discussion on T3034, we decided to replace SWHID with
    two or three classes:
    
    * QualifiedSWHID to replace the existing SWHID (standard types + qualifiers)
    * CoreSWHID, for "core SWHID" only (standard types + no qualifiers)
    * ExtendedSWHID for internal use in Software Heritage (extra types + no qualifiers)
    
    Since migrating from SWHID will break existing code, this commit uses
    the opportunity to modernize it a little, ie.:
    
    * `keyword`-only constructor, to get rid of the hacky default values for
      `object_type` and `object_id`
    * enum instead of strings for the object type
    * `bytes` instead of an hex string for the object id
    * rename `metadata` to `qualifiers`

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

Harbormaster completed remote builds in B19334: Diff 18237.Feb 18 2021, 1:12 PM
vlorentz updated this revision to Diff 18239.Feb 18 2021, 1:33 PM

improve QualifiedSWHID's docstring:

  • use doctest
  • fix outdated syntax
  • show example of from_string
vlorentz mentioned this in D5107: Add new class CoreSWHID as an alternative to SWHID/QualifiedSWHID.Feb 18 2021, 1:34 PM
swh-public-ci added a comment.Feb 18 2021, 1:34 PM

Build has FAILED

Patch application report for D5081 (id=18239)

Rebasing onto 0c16581283...

Current branch diff-target is up to date.
Changes applied before test
commit d5f996bfbb3885a4612978bce7d03ad0e49df393
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Tue Feb 16 13:33:13 2021 +0100

    Add new class QualifiedSWHID to replace SWHID, and deprecate the latter.
    
    Following the discussion on T3034, we decided to replace SWHID with
    two or three classes:
    
    * QualifiedSWHID to replace the existing SWHID (standard types + qualifiers)
    * CoreSWHID, for "core SWHID" only (standard types + no qualifiers)
    * ExtendedSWHID for internal use in Software Heritage (extra types + no qualifiers)
    
    Since migrating from SWHID will break existing code, this commit uses
    the opportunity to modernize it a little, ie.:
    
    * `keyword`-only constructor, to get rid of the hacky default values for
      `object_type` and `object_id`
    * enum instead of strings for the object type
    * `bytes` instead of an hex string for the object id
    * rename `metadata` to `qualifiers`

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

Harbormaster failed remote builds in B19336: Diff 18239!Feb 18 2021, 1:34 PM
swh-public-ci added a comment.Feb 18 2021, 1:38 PM

Build has FAILED

Patch application report for D5081 (id=18239)

Rebasing onto 0c16581283...

Current branch diff-target is up to date.
Changes applied before test
commit d5f996bfbb3885a4612978bce7d03ad0e49df393
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Tue Feb 16 13:33:13 2021 +0100

    Add new class QualifiedSWHID to replace SWHID, and deprecate the latter.
    
    Following the discussion on T3034, we decided to replace SWHID with
    two or three classes:
    
    * QualifiedSWHID to replace the existing SWHID (standard types + qualifiers)
    * CoreSWHID, for "core SWHID" only (standard types + no qualifiers)
    * ExtendedSWHID for internal use in Software Heritage (extra types + no qualifiers)
    
    Since migrating from SWHID will break existing code, this commit uses
    the opportunity to modernize it a little, ie.:
    
    * `keyword`-only constructor, to get rid of the hacky default values for
      `object_type` and `object_id`
    * enum instead of strings for the object type
    * `bytes` instead of an hex string for the object id
    * rename `metadata` to `qualifiers`

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

vlorentz updated this revision to Diff 18240.Feb 18 2021, 1:42 PM

fix tox

swh-public-ci added a comment.Feb 18 2021, 1:43 PM

Build is green

Patch application report for D5081 (id=18240)

Rebasing onto 0c16581283...

Current branch diff-target is up to date.
Changes applied before test
commit 335a91e4697999fbe63180a530ee7d031f994b9f
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Tue Feb 16 13:33:13 2021 +0100

    Add new class QualifiedSWHID to replace SWHID, and deprecate the latter.
    
    Following the discussion on T3034, we decided to replace SWHID with
    two or three classes:
    
    * QualifiedSWHID to replace the existing SWHID (standard types + qualifiers)
    * CoreSWHID, for "core SWHID" only (standard types + no qualifiers)
    * ExtendedSWHID for internal use in Software Heritage (extra types + no qualifiers)
    
    Since migrating from SWHID will break existing code, this commit uses
    the opportunity to modernize it a little, ie.:
    
    * `keyword`-only constructor, to get rid of the hacky default values for
      `object_type` and `object_id`
    * enum instead of strings for the object type
    * `bytes` instead of an hex string for the object id
    * rename `metadata` to `qualifiers`

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

Harbormaster completed remote builds in B19337: Diff 18240.Feb 18 2021, 1:43 PM
vlorentz updated this revision to Diff 18241.Feb 18 2021, 1:55 PM

fix typo in docstring

swh-public-ci added a comment.Feb 18 2021, 1:56 PM

Build is green

Patch application report for D5081 (id=18241)

Rebasing onto 0c16581283...

Current branch diff-target is up to date.
Changes applied before test
commit c7161592c48c6853c99a453ac079170fed7ee451
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Tue Feb 16 13:33:13 2021 +0100

    Add new class QualifiedSWHID to replace SWHID, and deprecate the latter.
    
    Following the discussion on T3034, we decided to replace SWHID with
    two or three classes:
    
    * QualifiedSWHID to replace the existing SWHID (standard types + qualifiers)
    * CoreSWHID, for "core SWHID" only (standard types + no qualifiers)
    * ExtendedSWHID for internal use in Software Heritage (extra types + no qualifiers)
    
    Since migrating from SWHID will break existing code, this commit uses
    the opportunity to modernize it a little, ie.:
    
    * `keyword`-only constructor, to get rid of the hacky default values for
      `object_type` and `object_id`
    * enum instead of strings for the object type
    * `bytes` instead of an hex string for the object id
    * rename `metadata` to `qualifiers`

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

Harbormaster completed remote builds in B19338: Diff 18241.Feb 18 2021, 1:56 PM
vlorentz updated this revision to Diff 18291.Feb 19 2021, 10:49 AM

rebase + remove to_dict()

swh-public-ci added a comment.Feb 19 2021, 10:51 AM

Build is green

Patch application report for D5081 (id=18291)

Rebasing onto 758eb885d3...

Current branch diff-target is up to date.
Changes applied before test
commit 50a4eaff5b29439d900ba128d93f0cb075ecc60c
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Tue Feb 16 13:33:13 2021 +0100

    Add new class QualifiedSWHID to replace SWHID, and deprecate the latter.
    
    Following the discussion on T3034, we decided to replace SWHID with
    two or three classes:
    
    * QualifiedSWHID to replace the existing SWHID (standard types + qualifiers)
    * CoreSWHID, for "core SWHID" only (standard types + no qualifiers)
    * ExtendedSWHID for internal use in Software Heritage (extra types + no qualifiers)
    
    Since migrating from SWHID will break existing code, this commit uses
    the opportunity to modernize it a little, ie.:
    
    * `keyword`-only constructor, to get rid of the hacky default values for
      `object_type` and `object_id`
    * enum instead of strings for the object type
    * `bytes` instead of an hex string for the object id
    * rename `metadata` to `qualifiers`

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

Harbormaster completed remote builds in B19377: Diff 18291.Feb 19 2021, 10:51 AM
vlorentz updated this revision to Diff 18300.Feb 19 2021, 1:50 PM

fix docstring

swh-public-ci added a comment.Feb 19 2021, 1:51 PM

Build is green

Patch application report for D5081 (id=18300)

Rebasing onto 758eb885d3...

Current branch diff-target is up to date.
Changes applied before test
commit 690b7f824f55bc65fbec21b29714926476038abb
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Tue Feb 16 13:33:13 2021 +0100

    Add new class QualifiedSWHID to replace SWHID, and deprecate the latter.
    
    Following the discussion on T3034, we decided to replace SWHID with
    two or three classes:
    
    * QualifiedSWHID to replace the existing SWHID (standard types + qualifiers)
    * CoreSWHID, for "core SWHID" only (standard types + no qualifiers)
    * ExtendedSWHID for internal use in Software Heritage (extra types + no qualifiers)
    
    Since migrating from SWHID will break existing code, this commit uses
    the opportunity to modernize it a little, ie.:
    
    * `keyword`-only constructor, to get rid of the hacky default values for
      `object_type` and `object_id`
    * enum instead of strings for the object type
    * `bytes` instead of an hex string for the object id
    * rename `metadata` to `qualifiers`

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

Harbormaster completed remote builds in B19386: Diff 18300.Feb 19 2021, 1:51 PM
vlorentz added a child revision: D5107: Add new class CoreSWHID as an alternative to SWHID/QualifiedSWHID.Feb 19 2021, 2:18 PM
ardumont accepted this revision.Feb 22 2021, 3:20 PM
ardumont added inline comments.
swh/model/tests/test_identifiers.py
1203

what's the _x ? (it's not immediatly apparent to me).

This revision is now accepted and ready to land.Feb 22 2021, 3:20 PM
ardumont added a comment.Feb 22 2021, 3:27 PM

Since migrating from SWHID will break existing code

Do you also plan on migrating the code which will break following that release?

vlorentz added a comment.Feb 23 2021, 9:44 AM
In D5081#129601, @ardumont wrote:

Since migrating from SWHID will break existing code

Do you also plan on migrating the code which will break following that release?

yes

swh/model/tests/test_identifiers.py
1203

hash_to_bytes. it's used everywhere in this module so it makes sense to have a short alias

ardumont added inline comments.Feb 23 2021, 9:46 AM
swh/model/tests/test_identifiers.py
1203

lol, i noticed it after writing the comment and removed the comment (or so i thought)...
sorry for the noise.

Closed by commit rDMOD690b7f824f55: Add new class QualifiedSWHID to replace SWHID, and deprecate the latter. (authored by vlorentz). · Explain WhyFeb 25 2021, 3:28 PM
This revision was automatically updated to reflect the committed changes.
vlorentz added a commit: rDMOD690b7f824f55: Add new class QualifiedSWHID to replace SWHID, and deprecate the latter..

Revision Contents
Changeset List

PathSize
swh/
model/
141 lines
tests/
261 lines
1 line

Diff 18393

View Options

swh/model/identifiers.py

Loading...
View Options

swh/model/tests/test_identifiers.py

Loading...
View Options

tox.ini

Loading...
About · Developer information · Legal