Skip to content

Commit

Permalink
Bump minrelaytxfee default
Browse files Browse the repository at this point in the history
To bridge the time until a dynamic method for determining this fee is
merged.

This is especially aimed at the stable releases (0.10, 0.11) because
full mempool limiting, as will be in 0.12, is too invasive and risky to
backport.
  • Loading branch information
laanwj committed Oct 9, 2015
1 parent 8c7e613 commit 28e3249
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/main.cpp
Expand Up @@ -75,7 +75,7 @@ uint64_t nPruneTarget = 0;
bool fAlerts = DEFAULT_ALERTS;

/** Fees smaller than this (in satoshi) are considered zero fee (for relaying and mining) */
CFeeRate minRelayTxFee = CFeeRate(1000);
CFeeRate minRelayTxFee = CFeeRate(5000);

CTxMemPool mempool(::minRelayTxFee);

Expand Down

1 comment on commit 28e3249

@schildbach
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What's the reason for this bump? Even in face of the current spam attack, 2000 sat./KB is enough to get txns confirmed within a couple of blocks. I tested this using bitcoinj / TestFeeLevel.java.

Please sign in to comment.