- they are internal to the DB so they do not belong in Kafka
- on unrelated errors, they cause swh.journal to crash because it does not know how to handle integers in the output of unique_key()
Closes T4719
Details
Details
- Reviewers
ardumont - Group Reviewers
Reviewers - Maniphest Tasks
- T4719: indexer storage crashes on kafka errors, because of integers in the key
- Commits
- rDCIDXe8549400bc54: Remove tool ids from Kafka messages
Diff Detail
Diff Detail
- Repository
- rDCIDX Metadata indexer
- Branch
- master
- Lint
No Linters Available - Unit
No Unit Test Coverage - Build Status
Buildable 33146 Build 51958: Phabricator diff pipeline on jenkins Jenkins console · Jenkins Build 51957: arc lint + arc unit
Event Timeline
Comment Actions
Build has FAILED
Patch application report for D8934 (id=32192)
Rebasing onto f74b47bcf4...
Current branch diff-target is up to date.
Changes applied before test
commit d9806be469102e81621493ce596c1a15680c5a4c
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date: Wed Dec 7 14:06:36 2022 +0100
Remove tool ids from Kafka messages
1. they are internal to the DB so they do not belong in Kafka
2. on unrelated errors, they cause swh.journal to crash because it does
not know how to handle integers in the output of unique_key()Link to build: https://jenkins.softwareheritage.org/job/DCIDX/job/tests-on-diff/535/
See console output for more information: https://jenkins.softwareheritage.org/job/DCIDX/job/tests-on-diff/535/console
Comment Actions
Build is green
Patch application report for D8934 (id=32195)
Rebasing onto f74b47bcf4...
Current branch diff-target is up to date.
Changes applied before test
commit e8549400bc547cff769671e945566f5343aa320f
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date: Wed Dec 7 14:06:36 2022 +0100
Remove tool ids from Kafka messages
1. they are internal to the DB so they do not belong in Kafka
2. on unrelated errors, they cause swh.journal to crash because it does
not know how to handle integers in the output of unique_key()See https://jenkins.softwareheritage.org/job/DCIDX/job/tests-on-diff/536/ for more details.