[ 
https://issues.apache.org/jira/browse/SOLR-8621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15144869#comment-15144869
 ] 

Christine Poerschke commented on SOLR-8621:
-------------------------------------------

Err, oops, having done that master/branch_5x/branch_5_5 consistency check above 
and looked at the 
[fe2cf250796a98ff1791a504d21acb67f0a1c397|http://git-wip-us.apache.org/repos/asf/lucene-solr/commit/fe2cf250796a98ff1791a504d21acb67f0a1c397]
 commit again, I think we need one more little tweak:
{code}
     if (mergePolicyInfo != null) {
       m.put("mergePolicy", mergePolicyInfo.toMap());
     } else if (mergePolicyFactoryInfo != null) {
-      m.put("mergePolicy", mergePolicyFactoryInfo.toMap());
+      m.put("mergePolicyFactory", mergePolicyFactoryInfo.toMap());
     }
{code}
with likely an accompanying test code change.

> solrconfig.xml: deprecate/replace <mergePolicy> with <mergePolicyFactory>
> -------------------------------------------------------------------------
>
>                 Key: SOLR-8621
>                 URL: https://issues.apache.org/jira/browse/SOLR-8621
>             Project: Solr
>          Issue Type: Task
>            Reporter: Christine Poerschke
>            Assignee: Christine Poerschke
>             Fix For: 5.5, master
>
>         Attachments: SOLR-8621-example_contrib_configs.patch, 
> SOLR-8621-example_contrib_configs.patch, SOLR-8621.patch, 
> explicit-merge-auto-set.patch
>
>
> *<mergePolicyFactory> end-user benefits:*
> * Lucene's UpgradeIndexMergePolicy can be configured in Solr
> * Lucene's SortingMergePolicy can be configured in Solr (with SOLR-5730)
> * customisability: arbitrary merge policies including wrapping/nested merge 
> policies can be created and configured
> *roadmap:*
> * solr 5.5 introduces <mergePolicyFactory> support
> * solr 5.5 deprecates (but maintains) <mergePolicy> support
> * SOLR-8668 in solr 6.0(\?) will remove <mergePolicy> support 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to