Xapian 1.4.5 "Db block overwritten - are there multiple writers?" with Glass

[Olly Betts]

On Wed, Mar 07, 2018 at 08:16:23PM +0000, Olly Betts wrote:
> Just to update the status of this - I now have a C++ reproducer, and
> have found that the transaction isn't needed to reproduce this.  It
> also still reproduces when run under eatmydata [1], and together these
> bring the time to reproduce down to 4-5 minutes.  Still longer than
> ideal, but quick enough to start making some progress on narrowing
> down what's happening.

I was intending to finish analysing this and address it for 1.4.6, but
once I found the MSet::snippet() bug I wanted to get a fix for that out
promptly.

But I've dug into this some more now.  It's a cursor on the postlist
table which is used to look up document values which causes the problem.

The attached patch reset this cursor each time commit() is called, and
that fixes my C++ reproducer, though I think this ought to work as-is
and the real bug is at a lower level.  But if you're suffering with this
bug, the patch will probably help and there's not much scope for it
making things worse (this cursor member is lazily created, so if it's
NULL when needed a new cursor is created).

Cheers,
    Olly
-------------- next part --------------
A non-text attachment was scrubbed...
Name: glass-value-cursor-reset.patch
Type: text/x-diff
Size: 363 bytes
Desc: not available
URL: <http://lists.xapian.org/pipermail/xapian-discuss/attachments/20180709/25f2c2d1/attachment.patch>