[
https://issues.apache.org/jira/browse/LUCENE-3995?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13255961#comment-13255961
]
Dawid Weiss commented on LUCENE-3995:
-------------------------------------
Note to myself - this also affectes test coverage because it reduces static
context entropy (as pointed by Robert, Uwe).
> In LuceneTestCase.beforeClass, make a new random (also using the class
> hashcode) to vary defaults
> -------------------------------------------------------------------------------------------------
>
> Key: LUCENE-3995
> URL: https://issues.apache.org/jira/browse/LUCENE-3995
> Project: Lucene - Java
> Issue Type: Improvement
> Components: general/test
> Affects Versions: 4.0
> Reporter: Robert Muir
> Assignee: Dawid Weiss
>
> In LuceneTestCase, we set many static defaults like:
> * default codec
> * default infostream impl
> * default locale
> * default timezone
> * default similarity
> Currently each test run gets a single seed for the run, which means for
> example across one test run
> every single test will have say, SimpleText + infostream=off + Locale=german
> + timezone=EDT + similarity=BM25
> Because of that, we lose lots of basic mixed coverage across tests, and it
> also means the unfortunate
> individual who gets SimpleText or other slow options gets a REALLY SLOW test
> run, rather than amortizing
> this across all test runs.
> We should at least make a new random (getRandom() ^ className.hashCode()) to
> fix this so it works like before,
> but unfortunately that only fixes it for LuceneTestCase.
> Won't any subclasses that make random decisions in @BeforeClass (and we have
> many) still have the same problem?
> Maybe RandomizedRunner can instead be improved here?
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
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]
