[ https://issues.apache.org/jira/browse/SOLR-8621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15134365#comment-15134365 ]
Shai Erera commented on SOLR-8621: ---------------------------------- Thanks [~cpoerschke], I'll review the two commits that you pushed. One quick comment/question that I have is about {{SolrIndexConfig}}'s defaultMP class and factory names. (1) I think they can be private since they're used only in {{SolrIndexConfig}}. Two, I think we can use an instance, rather than class name? So that in {{buildMPFromInfo}} we just create a new instance (i.e. {{new DefaultMPFactory().getMP()}}). What do you think? I'll give it another review later. > 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 > > *<mergePolicyFactory> end-user benefits:* > * Lucene's UpgradeIndexMergePolicy can be configured in Solr > * (with SOLR-5730) Lucene's SortingMergePolicy can be configured in Solr > * customisability: arbitrary merge policies including wrapping/nested merge > policies can be created and configured > *(proposed) roadmap:* > * solr 5.5 introduces <mergePolicyFactory> support > * solr 5.5(\?) deprecates (but maintains) <mergePolicy> support > * solr 6.0(\?) removes <mergePolicy> support > +work-in-progress git branch:+ > [master-solr-8621|https://github.com/apache/lucene-solr/tree/master-solr-8621] -- This message was sent by Atlassian JIRA (v6.3.4#6332) --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org