Olly Betts
2007-Apr-09 11:43 UTC
[Xapian-devel] Re: [Xapian-commits] 8153: trunk/xapian-core/ trunk/xapian-core/backends/flint/
On Mon, Apr 09, 2007 at 11:26:44AM +0100, richard wrote:> backends/flint/flint_database.cc: Fix bug with document lengths > in replace document. Was using doclens.insert() to set the new > document length, but this has no effect if the entry already > exists.Argh! That issue has bitten us at least once before. I'll audit all uses of insert to see if there are any other instances, unless you already have/are intending to. Cheers, Olly
Richard Boulton
2007-Apr-09 12:02 UTC
[Xapian-devel] Re: [Xapian-commits] 8153: trunk/xapian-core/ trunk/xapian-core/backends/flint/
Olly Betts wrote:> Argh! That issue has bitten us at least once before. I'll audit all > uses of insert to see if there are any other instances, unless you > already have/are intending to.No, I've not done this and hadn't planned to do so, but it's an excellent plan. I was thinking about working on the xapian-check to make it check document lengths in the postlists for consistency. -- Richard
Seemingly Similar Threads
- Re: [Xapian-commits] 8157: trunk/xapian-core/ trunk/xapian-core/backends/flint/ trunk/xapian-core/backends/quartz/
- Re: [Xapian-commits] 8351: trunk/xapian-core/ trunk/xapian-core/backends/flint/
- Re: [Xapian-commits] 7603: trunk/xapian-core/ trunk/xapian-core/backends/flint/ trunk/xapian-core/backends/quartz/
- Re: [Xapian-commits] 7603: trunk/xapian-core/trunk/xapian-core/backends/flint/ trunk/xapian-core/backends/quartz/
- Deleted documents not deleted