bitshares1-core icon indicating copy to clipboard operation
bitshares1-core copied to clipboard

Configurable client propagation parameters

Open hackfisher opened this issue 10 years ago • 7 comments

Refer another api bitcoin_settrxfee.

_chain_db->set_priority_fee( amount );

hackfisher avatar Jul 14 '14 21:07 hackfisher

We probably want a separate command like: wallet_delegate_set_min_priority_fee

vikramrajkumar avatar Jul 28 '14 01:07 vikramrajkumar

So what exactly is the issue here? We need a new API call to set the minimum fee for transactions my delegate(s) will include in their blocks?

nathanielhourt avatar Aug 07 '14 18:08 nathanielhourt

Progress: https://github.com/BitShares/bitshares_toolkit/commit/d17ac54b894621e752f2a093d9514beadcd44bae

vikramrajkumar avatar Aug 18 '14 14:08 vikramrajkumar

Does @nathanhourt describe this issue accurately? Please specify what the two fees are.

drltc avatar Oct 22 '14 22:10 drltc

The default fee to construct new transactions with can already be set with wallet_set_transaction_fee.

We should have another command that lets users set the minimum transaction fee that they require to propagate the transaction. This is currently set to 0.1 XTS and users are not able to change it.

vikramrajkumar avatar Oct 22 '14 23:10 vikramrajkumar

Instead of having a minimum prop fee set by each peer, we should have each delegate publish a TITAN profile field called min_prop_fee describing the smallest tx fee that delegate is willing to include in a block. Then peers set peer_min_prop_fee = min(m, 0.1 XTS) where m is the ~~minimum~~ median value of the delegates' min_prop_fee fields (or zero if no delegates currently publish that field).

The main motivation for this design is:

  • Allow clients to predict the effect of raising or lowering transaction fees
  • Transparently publish changes to the network's propagation behavior
  • Allow delegates to collectively control fees in a decentralized manner without requiring hardforking or requiring delegates to coordinate to all make a change at the same time

I think @bytemaster proposed something very similar to this on the forum some time ago.

drltc avatar Oct 24 '14 18:10 drltc

Edit above: Replace "minimum" with "median"

drltc avatar Oct 24 '14 21:10 drltc