Page MenuHomePhabricator

Add UTXO commitment contruction from UTXO cursor
AbandonedPublic

Authored by tomtomtom7 on Feb 16 2018, 15:29.

Details

Reviewers
None
Group Reviewers
Restricted Owners Package(Owns No Changed Paths)
Restricted Project
Summary

This is used for initial upgrade. The first version is still sequential.

We may also want to use this for (pre-fast-sync) Initial Load. As this
means we will need to only add 50 million UTXOs instead of 1.2 billion.

Depends on D1074

Test Plan

Test included

Diff Detail

Event Timeline

tomtomtom7 created this revision.Feb 16 2018, 15:29
Owners added a reviewer: Restricted Owners Package.Feb 16 2018, 15:29
Herald added a reviewer: Restricted Project. · View Herald TranscriptFeb 16 2018, 15:29
tomtomtom7 updated this revision to Diff 3010.Feb 28 2018, 15:59

Change the combine constructor to an Add method.

This is more consistent with the way it is used.

tomtomtom7 abandoned this revision.Mar 6 2018, 14:09

This is no integrated in D1074

Herald added 1 blocking reviewer(s): Restricted Project. · View Herald TranscriptMar 6 2018, 14:09
matiu added a subscriber: matiu.Jun 12 2018, 17:54
matiu added inline comments.
src/test/utxocommit_tests.cpp
157

Dead code for the current count = 50000;

tomtomtom7 added inline comments.Jun 14 2018, 12:51
src/test/utxocommit_tests.cpp
157

This diff is abandoned.