Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sweep up dust inputs when there is room #1675

Closed
gmaxwell opened this issue Aug 14, 2012 · 7 comments
Closed

Sweep up dust inputs when there is room #1675

gmaxwell opened this issue Aug 14, 2012 · 7 comments

Comments

@gmaxwell
Copy link
Contributor

The coin selection should automatically sweep up excessive small / dust inputs in order to get them out of the txout set when there is room and the txn already has change. (e.g. excess priority before the txn loses freeness, or room before the next kiliobyte fee increment)

This can be done as a post-processing step after the existing selection.

Care should be taken to avoid reducing privacy by linking otherwise independent accounts. This can sometimes increase privacy by further disguising the values exchange.

@luke-jr
Copy link
Member

luke-jr commented Aug 14, 2012

But fees might not be rounded to kilobyte (and aren't any more, with Gavin's changes, AFAIK)

@gmaxwell
Copy link
Contributor Author

Er. I'd missed that. I think we should keep the rounding behavior, both for improved compatibility with current behavior, and also because it leaves some free room for housekeeping operations like this.

@NotBugger
Copy link

@RayDillinger
Copy link

I would say that it is more important to structure costs correctly. It is the creation of dust txouts that causes both the expense of storing them on the blockchain and the eventual expense of redeeming them. A better solution IMO would be to make txins free and calculate fees based on txouts, and then allow clients sweep all dust from their accounts at will.

@gmaxwell
Copy link
Contributor Author

@Cryddit alas, for a miner, making them free would be economically irrational since the scarcity creating the market to begin with is the maximum size of the block.

(Also, making txins free entirely has the risk of people stuffing kilobytes of data in them for parasitic storage)

@maflcko
Copy link
Member

maflcko commented Dec 21, 2015

@gmaxwell This can be closed. We finally got rid of rounding up to the next 1kB in #7103 (rebase of #6708) and it's really unlikely those changes get reverted.

suprnurd pushed a commit to chaincoin-legacy/chaincoin that referenced this issue Dec 5, 2017
* move bitcoin release notes

* 12.2 release notes draft

* update

* update

* add latest commits, turn commits into links

* typo + 1704

* +1707

* +1711

* clarify 8x tx
@Empact
Copy link
Member

Empact commented Feb 16, 2018

@MarcoFalke I suppose you can close this now that you're a member?

@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

7 participants