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

Hoss Man updated SOLR-4951:
---------------------------

    Attachment: SOLR-4951.patch

here's a patch showing what i had in mind.

Some of the final getter/setter methods in MergePolicy make writing a true 
proxy class challenging, but i think this works out well enough, and i included 
some reflection based tests to try and help future proof against the risk of 
changes being made to the API that result in the class not behaving the same as 
whatever random impl it wraps.
                
> randomize merge policy testing in solr
> --------------------------------------
>
>                 Key: SOLR-4951
>                 URL: https://issues.apache.org/jira/browse/SOLR-4951
>             Project: Solr
>          Issue Type: Sub-task
>            Reporter: Hoss Man
>         Attachments: SOLR-4951.patch
>
>
> split off from SOLR-4942...
> * add a new RandomMergePolicy that implements MergePolicy by proxying to 
> another instance selected at creation using one of the 
> LuceneTestCase.new...MergePolicy methods
> * updated test configs to refer to this new MergePolicy
> * borrow the "tests.shardhandler.randomSeed" logic in SolrTestCaseJ4 to give 
> our RandomMergePolicy a consistent seed at runtime.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

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

Reply via email to