lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Adrien Grand <jpou...@gmail.com>
Subject Re: [JENKINS-EA] Lucene-Solr-trunk-Linux (64bit/jdk1.9.0-ea-b60)
Date Thu, 06 Aug 2015 20:33:20 GMT
+1

On Thu, Aug 6, 2015 at 10:29 PM, Christine Poerschke (BLOOMBERG/
LONDON) <cpoerschke@bloomberg.net> wrote:
> Hi Adrien,
>
> No, it was just 2 documents in 1 segment yesterday. Just returned to this and yes, you're
right, placing the 2nd document into a 2nd segment (prior to the force merge) seems to work,
at least with the seed that previously caused the test to fail.
>
> If that sounds about right, i'd be happy to properly patch for this tomorrow.
>
> Thanks,
> Christine
>
> --- a/lucene/misc/src/test/org/apache/lucene/search/TestEarlyTerminatingSortingCollector.java
> +++ b/lucene/misc/src/test/org/apache/lucene/search/TestEarlyTerminatingSortingCollector.java
> @@ -115,6 +115,8 @@ public class TestEarlyTerminatingSortingCollector extends LuceneTestCase
{
>        // the index, although want want a sorted segment so it needs to be merged
>        iw.getReader().close(); // refresh
>        iw.addDocument(new Document());
> +      iw.commit();
> +      iw.addDocument(new Document());
>        iw.forceMerge(1);
>      }
>      else if (random().nextBoolean()) {
>
> ----- Original Message -----
> From: dev@lucene.apache.org
> To: Christine Poerschke (BLOOMBERG/ LONDON), dev@lucene.apache.org
> At: Aug  6 2015 10:27:38
>
> Hi Christine,
>
> On Wed, Aug 5, 2015 at 11:42 PM, Christine Poerschke (BLOOMBERG/
> LONDON) <cpoerschke@bloomberg.net> wrote:
>> Locally changed createRandomIndex to add not the 1 but 2 documents before the forceMerge.
Still ends up with an unsorted segment:
>
> Did you ensure that there is a reopen between these two documents are
> added, so that we not only have 2 docs but also 2 segments? This can
> be done eg. by calling iw.getReader().close(); on the
> RandomIndexWriter.
>
> --
> Adrien
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@lucene.apache.org
> For additional commands, e-mail: dev-help@lucene.apache.org
>
>



-- 
Adrien

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@lucene.apache.org
For additional commands, e-mail: dev-help@lucene.apache.org


Mime
View raw message