]> Sergey Matveev's repositories - public-inbox.git/commit
searchidx: do not modify Xapian DB while iterating
authorEric Wong (Contractor, The Linux Foundation) <e@80x24.org>
Tue, 27 Feb 2018 20:25:23 +0000 (20:25 +0000)
committerEric Wong (Contractor, The Linux Foundation) <e@80x24.org>
Wed, 28 Feb 2018 23:06:20 +0000 (23:06 +0000)
commit44afe76a0a351e350e18a94876fc30edc4adc404
tree079ca13cb9d8c1277b3c699f68cef107b71760cd
parent79e6a74dbc6274efe6a7150f97bdac3043de42cf
searchidx: do not modify Xapian DB while iterating

Iterating through a list of documents while modifying them does
not seem to be supported in Xapian and it can trigger
DatabaseCorruptError exceptions.  This only worked with past
datasets out of dumb luck.  With the work-in-progress "v2"
public-inbox layout, this problem might become more visible
as the "thread skeleton" is partitioned out to a separate,
smaller Xapian database.

I've reproduced the problem on both Debian 8.x and 9.x with
Xapian 1.2.19 (chert backend) and 1.4.3 (glass backend)
respectively.
lib/PublicInbox/SearchIdx.pm