Page MenuHomePhabricator
Differential D1480

Tweak UTXO weight based agreed suggested costs
AbandonedPublic
Actions

Authored by schancel on Jun 6 2018, 03:17.

Details

Reviewers
jasonbcox
Group Reviewers
Restricted Project
Summary

Increase UTXO additional bytes by one order of magnitude. Decreases cost to 0.25 sat/byte for standard 2x2 transactions or any similar NxN transaction. However, creating a UTXO is now about 13 times as expensive. (34 bytes at 1 sat/byte vs 1824 @ 0.25 sat/byte)

Depends on D1486

Test Plan
make check && ./test/functional/test_runner.py

Diff Detail

Repository
rABC Bitcoin ABC
Branch
adjust-weight
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 2998
Build 4090: Bitcoin ABC Buildbot (legacy)
Build 4089: arc lint + arc unit

Event Timeline

schancel created this revision.Jun 6 2018, 03:17
Herald added a reviewer: Restricted Project. · View Herald TranscriptJun 6 2018, 03:17
teamcity added a subscriber: teamcity.Jun 6 2018, 03:18

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

teamcity added a comment.Jun 6 2018, 03:53

Build Bitcoin-ABC :: Diffs :: Bitcoin-ABC Diff Testing failed.

Harbormaster failed remote builds in B2706: Diff 4070!Jun 6 2018, 03:53
jasonbcox requested changes to this revision.Jun 8 2018, 23:10
jasonbcox added a subscriber: jasonbcox.

Can you indicate that this diff depends on D1486 in the summary?

This revision now requires changes to proceed.Jun 8 2018, 23:10
jasonbcox mentioned this in D1486: Use a modified size for transaction minimum fee calculation..Jun 8 2018, 23:26
schancel edited the summary of this revision. (Show Details)Jul 2 2018, 17:36
schancel added a parent revision: D1486: Use a modified size for transaction minimum fee calculation..
schancel updated this revision to Diff 4356.Jul 18 2018, 21:29

Rebase with lower fee

teamcity added a comment.Jul 18 2018, 21:31

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

schancel edited the summary of this revision. (Show Details)Jul 18 2018, 21:32
jasonbcox accepted this revision.Jul 18 2018, 21:40
This revision is now accepted and ready to land.Jul 18 2018, 21:40
teamcity added a comment.Jul 18 2018, 22:09

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

Harbormaster failed remote builds in B2906: Diff 4356!Jul 18 2018, 22:09
schancel edited parent revisions, added: D1581: Update CTxMemPoolEntry and BlockAssembler to billing cutoffs; removed: D1486: Use a modified size for transaction minimum fee calculation..Jul 19 2018, 20:28
schancel updated this revision to Diff 4455.Jul 26 2018, 21:55
schancel added a parent revision: D1486: Use a modified size for transaction minimum fee calculation..

Rebase

teamcity added a comment.Jul 26 2018, 21:56

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

teamcity added a comment.Jul 26 2018, 22:37

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

Harbormaster failed remote builds in B2983: Diff 4455!Jul 26 2018, 22:38
schancel updated this revision to Diff 4463.Jul 27 2018, 00:51

Rebase on D1581

teamcity added a comment.Jul 27 2018, 00:53

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

Harbormaster failed remote builds in B2990: Diff 4463!Jul 27 2018, 01:29
teamcity added a comment.Jul 27 2018, 01:29

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

schancel updated this revision to Diff 4473.Jul 30 2018, 22:24

Rebase

teamcity added a comment.Jul 30 2018, 22:26

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

teamcity added a comment.Jul 30 2018, 23:05

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

Harbormaster failed remote builds in B2998: Diff 4473!Jul 30 2018, 23:06
schancel abandoned this revision.Sep 24 2018, 18:31

Revision Contents
Changeset List

PathSize
src/
6 lines
2 lines
2 lines
policy/
2 lines
primitives/
2 lines
test/
2 lines
2 lines
test/
functional/
test_framework/
2 lines

Diff 4473

View Options

src/init.cpp

Loading...
View Options

src/miner.cpp

Loading...
View Options

src/net_processing.cpp

Loading...
View Options

src/policy/policy.h

Loading...
View Options

src/primitives/transaction.cpp

Loading...
View Options

src/test/miner_tests.cpp

Loading...
View Options

src/validation.h

Loading...
View Options

test/functional/test_framework/mininode.py

Loading...