Re: svn commit: r1503837 - /lucene/dev/trunk/lucene/queries/src/test/org/apache/lucene/queries/TermFilterTest.java

2013-07-16 Thread Uwe Schindler
Hi Martijn, Maybe remove the System.out? Uwe m...@apache.org schrieb: Author: mvg Date: Tue Jul 16 19:15:24 2013 New Revision: 1503837 URL: http://svn.apache.org/r1503837 Log: LUCENE-5088: Fixed test Modified:

Re: svn commit: r1503837 - /lucene/dev/trunk/lucene/queries/src/test/org/apache/lucene/queries/TermFilterTest.java

2013-07-16 Thread Martijn v Groningen
Just did this... On 16 July 2013 21:19, Uwe Schindler u...@thetaphi.de wrote: Hi Martijn, Maybe remove the System.out? Uwe m...@apache.org schrieb: Author: mvg Date: Tue Jul 16 19:15:24 2013 New Revision: 1503837 URL: http://svn.apache.org/r1503837 Log: LUCENE-5088: Fixed test

Re: svn commit: r1503837 - /lucene/dev/trunk/lucene/queries/src/test/org/apache/lucene/queries/TermFilterTest.java

2013-07-16 Thread Robert Muir
Does this really fix this or just make it less likely to happen...? On Tue, Jul 16, 2013 at 12:15 PM, m...@apache.org wrote: String value1 = _TestUtil.randomRealisticUnicodeString(random()); - String value2 = _TestUtil.randomRealisticUnicodeString(random()); + String value2

Re: svn commit: r1503837 - /lucene/dev/trunk/lucene/queries/src/test/org/apache/lucene/queries/TermFilterTest.java

2013-07-16 Thread Martijn v Groningen
This should fix this, b/c value2 (one extra character) will always be different than value1. The test initially failed b/c value1 and value2 were equal. On 16 July 2013 21:37, Robert Muir rcm...@gmail.com wrote: Does this really fix this or just make it less likely to happen...? On Tue, Jul