bench: bugfix, disable birth time block skip for wallet_create_tx.cpp #27907

pull furszy wants to merge 1 commits into bitcoin:master from furszy:2023_bench_fix_wallet_create_tx changing 1 files +4 −0
  1. furszy commented at 11:47 PM on June 16, 2023: member

    As the benchmarks inside wallet_create_tx.cpp assert the wallet balance at the end, they require all blocks to be scanned by the wallet. So, we need to ensure that no blocks are skipped by the recently added wallet birth time functionality.

    This just means setting the wallet birth time to the genesis block time. So the wallet is always older than any new block.

  2. DrahtBot commented at 11:47 PM on June 16, 2023: contributor

    <!--e57a25ab6845829454e8d69fc972939a-->

    The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

    <!--021abf342d371248e50ceaed478a90ca-->

    Reviews

    See the guideline for information on the review process.

    Type Reviewers
    ACK hernanmarino, TheCharlatan, achow101

    If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.

  3. DrahtBot added the label Tests on Jun 16, 2023
  4. furszy force-pushed on Jun 16, 2023
  5. bench: disable birth time block skip for wallet_create_tx.cpp
    As the benchmarks inside wallet_create_tx.cpp assert the
    wallet balance at the end, they require all
    blocks to be scanned by the wallet. So, we need
    to ensure that no blocks are skipped by the recently
    added wallet birth time functionality.
    
    This just means setting the wallet birthtime to the
    genesis block time. So the wallet is always older than
    any new block.
    a72af2e833
  6. furszy force-pushed on Jun 17, 2023
  7. hernanmarino approved
  8. hernanmarino commented at 3:37 AM on June 18, 2023: contributor

    ACK a72af2e833bcf7baa79a246609da90ae0ade2a82

  9. fanquake requested review from achow101 on Jun 20, 2023
  10. TheCharlatan approved
  11. TheCharlatan commented at 10:49 AM on June 20, 2023: contributor

    ACK a72af2e833bcf7baa79a246609da90ae0ade2a82

  12. achow101 commented at 5:40 PM on June 20, 2023: member

    ACK a72af2e833bcf7baa79a246609da90ae0ade2a82

  13. DrahtBot removed review request from achow101 on Jun 20, 2023
  14. achow101 merged this on Jun 20, 2023
  15. achow101 closed this on Jun 20, 2023

  16. sidhujag referenced this in commit a3279ab272 on Jun 21, 2023
  17. furszy deleted the branch on Aug 9, 2023
  18. bitcoin locked this on Aug 8, 2024

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-04-16 00:13 UTC

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