Page MenuHomePhabricator

[index] Create new TxIndex class.

Authored by markblundeberg on May 2 2019, 18:02.



backports four commits from core PR13033:
f90c3a62f [index] TxIndex method to wait until caught up.
70d510d93 [index] Allow TxIndex sync thread to be interrupted.
94b4f8bbb [index] TxIndex initial sync thread.
34d68bf3a [index] Create new TxIndex class.

Depends D2901

Test Plan

build (tests are available in a later commit but not compatible
with current FindTx definition)

Diff Detail

rABC Bitcoin ABC
Lint OK
No Unit Test Coverage
Build Status
Buildable 5649
Build 9360: Bitcoin ABC Buildbot (legacy)
Build 9359: arc lint + arc unit

Event Timeline

markblundeberg created this revision.May 2 2019, 18:02
Herald added a reviewer: Restricted Project. · View Herald TranscriptMay 2 2019, 18:02
markblundeberg updated this revision to Diff 8342.May 2 2019, 18:24

fix linting (had an exception imported from the future)

deadalnix requested changes to this revision.May 5 2019, 16:49

No space in PR12345. please fix description.

This revision now requires changes to proceed.May 5 2019, 16:49
markblundeberg requested review of this revision.May 5 2019, 16:57
markblundeberg edited the summary of this revision. (Show Details)

No space in PR12345. please fix description.

OK, will fix other diffs likewise.

You also need to bring test/txindex_tests.cpp with it.

You also need to bring test/txindex_tests.cpp with it.

Unfortunately the tests don't work here because FindTx gets redefined in a later commit (that only occurs after the 'activation' code)

deadalnix accepted this revision.May 16 2019, 17:02
This revision is now accepted and ready to land.May 16 2019, 17:02
This revision was automatically updated to reflect the committed changes.