Page MenuHomeSoftware Heritage

Remove concept of 'minimal set' of metadata.
ClosedPublic

Authored by vlorentz on Aug 9 2019, 2:05 PM.

Details

Summary

So: rename of 'extract_minimal_metadata_dict' to 'merge_documents'
and move to the relevant module.

I don't really understand the point of having this terms whitelist,
since each metadata mapping already is a de facto whitelist.

Diff Detail

Repository
rDCIDX Metadata 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.Aug 9 2019, 2:05 PM
ardumont accepted this revision.Aug 9 2019, 3:40 PM
ardumont added a subscriber: ardumont.

That seems simpler. Tests are green. Go!

This revision is now accepted and ready to land.Aug 9 2019, 3:40 PM
This revision was automatically updated to reflect the committed changes.