Page MenuHomeSoftware Heritage

Prevent OriginMetadataIndexer from sending duplicate revisions to revision_metadata_add.
ClosedPublic

Authored by vlorentz on Tue, Feb 5, 12:04 PM.

Diff Detail

Repository
rDCIDX Object indexer
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

vlorentz created this revision.Tue, Feb 5, 12:04 PM
ardumont accepted this revision.Wed, Feb 6, 3:15 PM
ardumont added a subscriber: ardumont.
ardumont added inline comments.
swh/indexer/metadata.py
312

why not build the 2 lists (rev_metadata, orig_metadata) in one go since you iterate over it here already?

This revision is now accepted and ready to land.Wed, Feb 6, 3:15 PM
vlorentz updated this revision to Diff 3422.Wed, Feb 6, 5:06 PM
  • Simplify code
vlorentz updated this revision to Diff 3423.Wed, Feb 6, 5:07 PM
vlorentz marked 2 inline comments as done.
  • Fix
vlorentz added inline comments.Wed, Feb 6, 5:07 PM
swh/indexer/metadata.py
312

Good catch

This revision was automatically updated to reflect the committed changes.