dump: make dump take Xapian write lock
authorMark Walters <markwalters1009@gmail.com>
Mon, 23 Jun 2014 20:12:29 +0000 (21:12 +0100)
committerDavid Bremner <david@tethera.net>
Wed, 16 Jul 2014 22:33:10 +0000 (19:33 -0300)
commitbbbdf0478ea6109afc1ca5a26a8d1cd86b600e2a
tree4b1aef8c3f60dc6f7bb755002303b650677dad72
parent567aab405b05d9ef79ddf1b62302806b9ddbeaad
dump: make dump take Xapian write lock

Dump currently only takes the read lock. Xapian can cope with some
changes while maintaining a read snapshot but with more changes it
fails. Currently notmuch just gives a xapian error.

To avoid this we take the write lock when dumping. This prevents other
notmuch processes from modifying the xapian database preventing this
error.

Discussion with Olly on irc indicates that this is currently the best
solution: in xapian trunk there may be better possibilities using
snapshots but they need to make it to a release and propogate out to
users before we can switch approach.

Finally, this breaks one use case: pipelines of the form

notmuch dump | ... | notmuch restore

According to Olly this is already very fragile: it will only work on
small databases. One of the tests relies on this behaviour so fix that
to store the dump rather than use a pipe.
notmuch-dump.c
test/T150-tagging.sh