Benchmark sanity checks and fork checks in ConnectBlock #6965

pull TheBlueMatt wants to merge 1 commits into bitcoin:master from TheBlueMatt:bench changing 1 files +19 −9
  1. TheBlueMatt commented at 11:25 PM on November 6, 2015: contributor

    Add new fields to benchmarking ConnectBlock. In my very basic tests "Sanity checks" uses ~10x "Callbacks" and "Index writing" combined and "Fork Checks" uses about 1/3 the time of connecting transactions!

  2. Benchmark sanity checks and fork checks in ConnectBlock 77f1f59d12
  3. TheBlueMatt commented at 11:26 PM on November 6, 2015: contributor

    Note that the second should use much less time with #6931, though I havent tested it yet.

  4. TheBlueMatt cross-referenced this on Nov 6, 2015 from issue Skip BIP 30 verification where not necessary by morcos
  5. sipa commented at 1:29 AM on November 7, 2015: member

    Concept & code review ACK.

  6. gmaxwell commented at 1:42 AM on November 7, 2015: contributor

    ACK.

  7. laanwj added the label Validation on Nov 10, 2015
  8. laanwj merged this on Nov 10, 2015
  9. laanwj closed this on Nov 10, 2015

  10. laanwj referenced this in commit de7d4591a7 on Nov 10, 2015
  11. bitcoin locked this on Sep 8, 2021

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-05-20 06:55 UTC