Speed Up CuckooCache tests #10128

pull JeremyRubin wants to merge 1 commits into bitcoin:master from JeremyRubin:cuckoo-tests-faster changing 1 files +7 −6
  1. JeremyRubin commented at 7:34 pm on March 30, 2017: contributor

    I decreased all the sizes, should be much faster. Shouldn’t impact correctness of the cache or likelihood of catching bugs in the cache.

    Can be reverted if we want to do extended tests later.

    Addresses https://github.com/bitcoin/bitcoin/issues/10026

  2. Decrease testcase sizes in cuckoocache tests 3f098cccf6
  3. fanquake added the label Tests on Mar 30, 2017
  4. fanquake commented at 1:43 am on March 31, 2017: member

    Tested with src/test/test_bitcoin --log_level=all --run_test=cuckoocache_tests

    5 runs of master (edc62c959ab3ed7019b514107a4ab3865fc12c2e):

    0Leaving test suite "cuckoocache_tests"; testing time: 6987983us
    1Leaving test suite "cuckoocache_tests"; testing time: 6994218us
    2Leaving test suite "cuckoocache_tests"; testing time: 6903206us
    3Leaving test suite "cuckoocache_tests"; testing time: 7051359us
    4Leaving test suite "cuckoocache_tests"; testing time: 7055289us
    

    avg = ~6998411us

    vs 5 runs of this PR (3f098cc):

    0Leaving test suite "cuckoocache_tests"; testing time: 893674us
    1Leaving test suite "cuckoocache_tests"; testing time: 884415us
    2Leaving test suite "cuckoocache_tests"; testing time: 877087us
    3Leaving test suite "cuckoocache_tests"; testing time: 906258us
    4Leaving test suite "cuckoocache_tests"; testing time: 896657us
    

    avg = ~891618us

    = ~8times speedup

  5. gmaxwell approved
  6. gmaxwell commented at 7:37 pm on April 1, 2017: contributor
    utACK
  7. laanwj merged this on Apr 2, 2017
  8. laanwj closed this on Apr 2, 2017

  9. laanwj referenced this in commit 427d2fd04e on Apr 2, 2017
  10. PastaPastaPasta referenced this in commit 6a666bb300 on May 10, 2019
  11. PastaPastaPasta referenced this in commit b1905fba56 on May 15, 2019
  12. PastaPastaPasta referenced this in commit 818ec218d6 on May 20, 2019
  13. PastaPastaPasta referenced this in commit aae36f83b1 on May 21, 2019
  14. barrystyle referenced this in commit 4c4201ebb3 on Jan 22, 2020
  15. MarcoFalke locked this on Dec 16, 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: 2024-11-17 15:12 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me