You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@lucene.apache.org by "Ishan Chattopadhyaya (JIRA)" <ji...@apache.org> on 2017/03/07 13:26:38 UTC

[jira] [Updated] (SOLR-10178) TestInPlaceUpdatesDistrib unable to use NoMergePolicy[Factory] on branch_6x

     [ https://issues.apache.org/jira/browse/SOLR-10178?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Ishan Chattopadhyaya updated SOLR-10178:
----------------------------------------
    Description: 
TestInPlaceUpdatesDistrib depends on consistent segments to track docIds to assert in-place updates.

Towards that, NoMergePolicy is best suited and working fine on master (by defining it using NoMergePolicyFactory).

This doesn't work with just the MergePolicy (systemSetPropertySolrTestsMergePolicy()), since NoMergePolicy is a singleton and doesn't have a constructor. Setting only a NoMergePolicyFactory (systemSetPropertySolrTestsMergePolicyFactory()) seems to take no effect and falls back on RandomMergePolicyFactory. As a result, this test cannot use the NoMergePolicy[Factory].

Here's the corresponding test failure: https://jenkins.thetaphi.de/job/Lucene-Solr-master-Linux/19012/

-Seems to me that SOLR-8668 needs to be backported to branch_6x for this test to work, or some other stopgap hack needs to be put in place to make it work before SOLR-8668 is backported. [~cpoerschke], WDYT?-
Edit: This is also valid for master. ^

  was:
TestInPlaceUpdatesDistrib depends on consistent segments to track docIds to assert in-place updates.

Towards that, NoMergePolicy is best suited and working fine on master (by defining it using NoMergePolicyFactory).

This doesn't work with just the MergePolicy (systemSetPropertySolrTestsMergePolicy()), since NoMergePolicy is a singleton and doesn't have a constructor. Setting only a NoMergePolicyFactory (systemSetPropertySolrTestsMergePolicyFactory()) seems to take no effect and falls back on RandomMergePolicyFactory. As a result, this test cannot use the NoMergePolicy[Factory].

Here's the corresponding test failure: https://jenkins.thetaphi.de/job/Lucene-Solr-master-Linux/19012/

Seems to me that SOLR-8668 needs to be backported to branch_6x for this test to work, or some other stopgap hack needs to be put in place to make it work before SOLR-8668 is backported. [~cpoerschke], WDYT?


> TestInPlaceUpdatesDistrib unable to use NoMergePolicy[Factory] on branch_6x
> ---------------------------------------------------------------------------
>
>                 Key: SOLR-10178
>                 URL: https://issues.apache.org/jira/browse/SOLR-10178
>             Project: Solr
>          Issue Type: Test
>      Security Level: Public(Default Security Level. Issues are Public) 
>            Reporter: Ishan Chattopadhyaya
>
> TestInPlaceUpdatesDistrib depends on consistent segments to track docIds to assert in-place updates.
> Towards that, NoMergePolicy is best suited and working fine on master (by defining it using NoMergePolicyFactory).
> This doesn't work with just the MergePolicy (systemSetPropertySolrTestsMergePolicy()), since NoMergePolicy is a singleton and doesn't have a constructor. Setting only a NoMergePolicyFactory (systemSetPropertySolrTestsMergePolicyFactory()) seems to take no effect and falls back on RandomMergePolicyFactory. As a result, this test cannot use the NoMergePolicy[Factory].
> Here's the corresponding test failure: https://jenkins.thetaphi.de/job/Lucene-Solr-master-Linux/19012/
> -Seems to me that SOLR-8668 needs to be backported to branch_6x for this test to work, or some other stopgap hack needs to be put in place to make it work before SOLR-8668 is backported. [~cpoerschke], WDYT?-
> Edit: This is also valid for master. ^



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

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