To avoid lots of useless transactions.
Details
Details
- Reviewers
anlambert - Group Reviewers
Reviewers - Commits
- rDSCHcc76a573e795: journal_client: Only upsert if we have something to upsert
tox happy
Diff Detail
Diff Detail
- Repository
- rDSCH Scheduling utilities
- Branch
- master
- Lint
No Linters Available - Unit
No Unit Test Coverage - Build Status
Buildable 23182 Build 36171: Phabricator diff pipeline on jenkins Jenkins console · Jenkins Build 36170: arc lint + arc unit
Event Timeline
Comment Actions
Build is green
Patch application report for D6140 (id=22224)
Rebasing onto 506f78c827...
Current branch diff-target is up to date.
Changes applied before test
commit cc76a573e795c9001473858af3591ff66304c152 Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org> Date: Thu Aug 26 11:44:14 2021 +0200 journal_client: Only upsert if we have something to upsert
See https://jenkins.softwareheritage.org/job/DSCH/job/tests-on-diff/445/ for more details.
Comment Actions
Looks good to me. Could you add a test for this or is it too complicated ? The code is covered but not the introduced behavior,
Comment Actions
Looks good to me. Could you add a test for this or is it too complicated ? The code is covered but not the introduced behavior,
I'm not entirely sure on how to proceed for the test indeed.
But for sure, it's currently working as expected on the production patched swh-scheduler-journal-client (saatchi).
Comment Actions
Ack, let's forget that test then. Considering the changes, it does not worth it to lose time on it.