lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Martijn v Groningen <>
Subject Re: svn commit: r1363272 - /lucene/dev/trunk/lucene/test-framework/src/java/org/apache/lucene/util/
Date Mon, 23 Jul 2012 08:20:37 GMT
Thanks for also taking a look at the test failures!

On 20 July 2012 19:14, Robert Muir <> wrote:
> Hi Martinj: thanks for looking into this!
> I think have a better fix for these:
> the problem is actually in the AssertingAtomicReaders that
> AssertingDirectoryReader wraps its subreaders with.
> So I added the invisible-cache-key hack there, and removed it
> completely from LuceneTestCase.
Yes this was the problem. I think this fix wouldn't make hudson fail
again. Since the
FC insanity can't be detected.

During some test runs the AtomicReaderContext (in
Collector#setNextReader method) contains a
SlowCompositeReaderWrapper and this reader wraps another reader
something like this:
_0(5.0):C463 _1(5.0):C930 _2(5.0):C378 _3(5.0):C184))))

This used to cause the FC insanity test failures, but is this a
scenario that actually needs to be tested? Having a top level
readercontext in the setNextReader method seems like wrong usage to


To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message