Default to DB_TXN_WRITE_NOSYNC for all transactional operations #1360

pull jgarzik wants to merge 1 commits into bitcoin:master from jgarzik:datasync changing 2 files +2 −1

jgarzik commented at 5:27 AM on May 19, 2012: contributor
- This is safer than DB_TXN_NOSYNC, and does not appear to impact performance.
- Applying this to the dbenv is necessary to avoid many fdatasync(2) calls on db 5.x
- We carefully and thoroughly flush databases upon shutdown and other important events already.

Default to DB_TXN_WRITE_NOSYNC for all transactional operations

* This is safer than DB_TXN_NOSYNC, and does not appear to impact
  performance.
* Applying this to the dbenv is necessary to avoid many fdatasync(2)
  calls on db 5.x
* We carefully and thoroughly flush databases upon shutdown and
  other important events already.

394b9298fa

gavinandresen commented at 1:27 PM on May 19, 2012: contributor

ACK
sipa commented at 3:02 PM on May 19, 2012: member

No noticeable improvement here (bdb 5.1), but ACK.
jgarzik referenced this in commit cde87ee75c on May 19, 2012
jgarzik merged this on May 19, 2012
jgarzik closed this on May 19, 2012
coblee referenced this in commit c9be7437bf on Jul 17, 2012
suprnurd referenced this in commit 0bc6d92334 on Dec 5, 2017
lateminer referenced this in commit 905d04b2c6 on Jan 22, 2019
lateminer referenced this in commit aee230135f on May 6, 2020
DrahtBot locked this on Sep 8, 2021

Contributors

jgarzik

gavinandresen

sipa