Page MenuHomeSoftware Heritage

Better error message on content retrieval error + don't write translated_metadata=None to the idx storage.
ClosedPublic

Authored by vlorentz on Jan 15 2019, 2:27 PM.

Diff Detail

Repository
rDCIDX Metadata indexer
Branch
error-content-retrieval
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 3510
Build 4546: tox-on-jenkinsJenkins
Build 4545: arc lint + arc unit

Event Timeline

ardumont added a subscriber: ardumont.
ardumont added inline comments.
swh/indexer/metadata.py
79
%s", hashutil.hash_to_hex(id))
This revision now requires changes to proceed.Jan 15 2019, 2:32 PM
swh/indexer/metadata.py
79

i meant:

%s" % hashutil.hash_to_hex(id))
vlorentz added inline comments.
swh/indexer/metadata.py
79

Either way works

  • even better error message
  • If there was an error translating metadata, return no result.
vlorentz retitled this revision from Better error message on content retrieval error. to Better error message on content retrieval error + don't write translated_metadata=None to the idx storage..Jan 15 2019, 3:40 PM
This revision is now accepted and ready to land.Jan 15 2019, 4:55 PM
This revision was automatically updated to reflect the committed changes.