Differential D14983

[Chronik] Add a merge lock to avoid a race in the tests
ClosedPublic
Actions

Authored by Fabien on Dec 14 2023, 09:22.

Details

Reviewers

PiRK

Group Reviewers

Restricted Project

Commits

rABCbcffff883058: [Chronik] Add a merge lock to avoid a race in the tests

Summary

Avoid a race when several tests are testing merge errors, similar to what's done outside of tests.

Extracted from D14817 by @tobias_ruck.

Test Plan

./contrib/teamcity/build-configurations.py build-chronik

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

Fabien created this revision.Dec 14 2023, 09:22

Herald added a reviewer: Restricted Project. · View Herald TranscriptDec 14 2023, 09:22

Fabien requested review of this revision.Dec 14 2023, 09:22

Harbormaster completed remote builds in B25958: Diff 43611.Dec 14 2023, 09:42

Fabien added a child revision: D14815: [chronik] Disable the Write Ahead Log for rocksdb.Dec 14 2023, 14:02

PiRK accepted this revision.Dec 14 2023, 14:31

This revision is now accepted and ready to land.Dec 14 2023, 14:31

Closed by commit rABCbcffff883058: [Chronik] Add a merge lock to avoid a race in the tests (authored by tobias_ruck, committed by Fabien). · Explain WhyDec 14 2023, 19:33

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABCbcffff883058: [Chronik] Add a merge lock to avoid a race in the tests.

Revision Contents
Changeset List

Path

Size

chronik/

chronik-db/

src/

io/

group_history.rs

12 lines

merge.rs

12 lines

spent_by.rs

11 lines

Diff 43620

View Options

chronik/chronik-db/src/io/group_history.rs

View Options

chronik/chronik-db/src/io/merge.rs

View Options

chronik/chronik-db/src/io/spent_by.rs

[Chronik] Add a merge lock to avoid a race in the testsClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 43620

chronik/chronik-db/src/io/group_history.rs

chronik/chronik-db/src/io/merge.rs

chronik/chronik-db/src/io/spent_by.rs

[Chronik] Add a merge lock to avoid a race in the tests
ClosedPublic
Actions

Revision Contents
Changeset List