IRC meeting summary for 2016-06-23

Overview


Main topics

  • perceived validation slowdowns
  • Segwit

perceived validation slowdowns

background

Gmaxwell noticed very slow reindexing while testing with default dbcache, which got confirmed by sipa who saw similar behavior.

meeting comments

Sipa noticed very slow chainstate writes, however that might be because of his disk setup. We still need to figure out what exactly is behind this, but it’s a good idea to crank up the default value for dbcache and change how it’s allocated as currently a large part goes to leveldb caches, but further benchmarks are needed to find the best values.

Gmaxwell will test against a patched version of 0.12.1 which skips signature checking before block 295000 to see if there’s any regression or this is normal behavior.

These tests where done before compact blocks was merged, so it can’t be affacted by this. Leveldb hasn’t had any changes recently so the only way for leveldb to be the issue is if it has crossed some performance cliff.

Sipa notes the testing has been done with txindex enabled (maintains a full transaction index), and performance might be influenced by this.

meeting conclusion

  • Gmaxwell will benchmark different configurations: 0.12.1 vs master, txindex enabled/disabled, dbcache default/higher and with different cache splits.

Segwit

background

Developers are working on a soft fork to introduce segregated witness onto Bitcoin mainnet. Segregated witness (segwit) allows transaction signature data to be stored outside of the data hashed to produce transaction identifiers, removing all known forms of third-party malleability, allowing full nodes to compile the current UTXO set without downloading all signatures, and laying the groundwork for fraud proofs that can allow lightweight (SPV) clients to help enforce more of the consensus rules. The segwit soft fork also allows miners to substitute 1 byte of block space with 4 bytes of segwit data, increasing transaction capacity for wallets that use segwit. segregated witness BIPs: BIP141, BIP142, BIP143, BIP144 and BIP145

meeting comments

Segwit has been rebased after the compact blocks merge.

Sipa has been running compact blocks + segwit and doesn’t see an impact on memory usage.

Everyone is in favor of merging segwit. (was merged after the meeting)

Gmaxwell suggests we’d put out ‘testnet binaries’, which has testnet enabled by default, right away to get more people testing. Up to now segwit testing was mostly done by technical people that aren’t likely to be confused by things like status changes in the UI or whatnot.

The activation of segwit on testnet was a very useful testing exercise as it was in an environment with most nodes not upgraded.

meeting conclusion

  • Merge segwit
  • Release testnet-default/only binaries

Comic relief

wumpus      meeting time?
sipa        present
gmaxwell    past?

petertodd   sipa: re: segwit, has it been rebased?
sipa        petertodd: 12 times by now
CodeShark   lol
CodeShark   poor sipa
wumpus      sipa is getting carpal tunnel syndrome from rebasing

lightningbot   Meeting ended Thu Jun 23 19:49:58 2016 UTC. 
jtimon      oh, I think we forgot to make a joke, that's bad for the summaries :p

Participants

IRC nick Name/Nym
petertodd Peter Todd
sipa Pieter Wuille
gmaxwell Gregory Maxwell
wumpus Wladimir van der Laan
instagibbs Gregory Sanders
btcdrak BtcDrak
jtimon Jorge Timon
CodeShark Eric Lombrozo

Disclaimer

This summary was compiled without input from any of the participants in the discussion, so any errors are the fault of the summary author and not the discussion participants.


Show your support

Github