Details

Reviewers

Group Reviewers

Restricted Project

Maniphest Tasks

T617: Backport PR 13033 "Build txindex in parallel with validation"

Summary

backports PR14071:
faf4a9b674 qa: Stop txindex thread before calling destructor (MarcoFalke)
and PR15410:
fab6b07c16 test: txindex: interrupt threadGroup before calling destructor (MarcoFalke)

Depends on D2924

Test Plan

make check

Diff Detail

Repository

rABC Bitcoin ABC

Branch

txindextest_fix

Lint

Lint OK

Unit

No Unit Test Coverage

Build Status

Buildable 5675
Build 9412: Bitcoin ABC Teamcity Staging
Build 9411: arc lint + arc unit

Event Timeline

markblundeberg created this revision.May 5 2019, 01:37

Herald added a reviewer: Restricted Project. · View Herald TranscriptMay 5 2019, 01:37

Herald added a reviewer: deadalnix. · View Herald Transcript

Build Bitcoin-ABC / Diffs / Bitcoin-ABC Diff Testing started.

markblundeberg added a task: T617: Backport PR 13033 "Build txindex in parallel with validation".May 5 2019, 01:44

Build Bitcoin-ABC / Diffs / Bitcoin-ABC Diff Testing passed.

Harbormaster completed remote builds in B5675: Diff 8390.May 5 2019, 01:52

markblundeberg edited the summary of this revision. (Show Details)May 5 2019, 17:08

Please merge that with the test and the creation go TxIndex. We do not benefit from commiting faulty code and then fixing it.

This revision now requires changes to proceed.May 5 2019, 21:44

In D2925#69387, @deadalnix wrote:

Please merge that with the test and the creation go TxIndex. We do not benefit from commiting faulty code and then fixing it.

Aye aye, cap'n.

markblundeberg mentioned this in D2924: [txindex] Activate new transaction index code that runs in background.May 5 2019, 21:55

Diff	ID	Base	Description	Created	Lint	Unit
Base			Base
Diff 1	8390	4ff046c		May 5 2019, 01:37	★	★

Status	Author	Revision
Closed	markblundeberg	D2928 Simplify semantics of ChainStateFlushed callback
Closed	markblundeberg	D2927 scripted-diff: Rename SetBestChain callback ChainStateFlushed
Closed	markblundeberg	D2938 index: Move index DBs into index/ directory.
Closed	markblundeberg	D2937 MOVEONLY: Move BaseIndex to its own file.
Closed	markblundeberg	D2936 index: Generalize logged statements in BaseIndex.
Closed	markblundeberg	D2935 index: Extract logic from TxIndex into reusable base class.
Closed	markblundeberg	D2934 db: Make reusable base class for index databases.
Closed	markblundeberg	D2933 db: Remove obsolete methods from CBlockTreeDB.
Closed	markblundeberg	D2932 Support serialization as another type without casting
Closed	markblundeberg	D2931 Support deserializing into temporaries
Closed	markblundeberg	D2930 Merge READWRITEMANY into READWRITE
Closed	markblundeberg	D2929 Sanitize some wallet serialization
Closed	markblundeberg	D3115 Set InitMessage for txindex migration
Abandoned	markblundeberg	D2925 test: Stop txindex thread before calling destructor
Closed	markblundeberg	D2924 [txindex] Activate new transaction index code that runs in background
Closed	markblundeberg	D2904 [index] Create new TxIndex class.
Closed	markblundeberg	D2903 [db] Migration for txindex data to new, separate database.
Closed	markblundeberg	D2901 [db] Create separate database for txindex.

test: Stop txindex thread before calling destructor
AbandonedPublic
Actions

Details

Diff Detail

Event Timeline

Revision Contents
Changeset List

Diff 8390

src/test/txindex_tests.cpp

test: Stop txindex thread before calling destructorAbandonedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 8390

src/test/txindex_tests.cpp

test: Stop txindex thread before calling destructor
AbandonedPublic
Actions

Revision Contents
Changeset List