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.
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.
If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.
DrahtBot added the label Tests on Jun 16, 2023
furszy force-pushed on Jun 16, 2023
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
furszy force-pushed on Jun 17, 2023
hernanmarino approved
hernanmarino
commented at 3:37 AM on June 18, 2023:
contributor
ACKa72af2e833bcf7baa79a246609da90ae0ade2a82
fanquake requested review from achow101 on Jun 20, 2023
TheCharlatan approved
TheCharlatan
commented at 10:49 AM on June 20, 2023:
contributor
ACKa72af2e833bcf7baa79a246609da90ae0ade2a82
achow101
commented at 5:40 PM on June 20, 2023:
member
ACKa72af2e833bcf7baa79a246609da90ae0ade2a82
DrahtBot removed review request from achow101 on Jun 20, 2023
achow101 merged this on Jun 20, 2023
achow101 closed this on Jun 20, 2023
sidhujag referenced this in commit a3279ab272 on Jun 21, 2023
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