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

Hoss Man commented on LUCENE-6194:
----------------------------------

bq. Can we just remove tests.iters already now that we have 'ant beast'?

-0

If you understand what you are doing, tests.iters can be really handy -- 
particularly in cases where you have already figured out that a test failure 
only happens with certain master seeds, but the failures are timing/threading 
related and you want to do a lot of iterations.  using tests.iters lets you 
re-use the static initialization of the test and only loop over the test method 
itself.

yes you can still re-run the same master seed over and over with beast, but 
you're doing a lot more teardown/setup and it takes a lot longer to do the same 
number of iterations.

if folks want to remove tests.iters, i suggest bring that up for discussion in 
it's own issue -- independent of if/when that is done, i still think the crux 
of this issue is relevant: there is no down side to includin the full seed in 
the reproduce line.


> Reproduce with should include -Dtests.iters=N
> ---------------------------------------------
>
>                 Key: LUCENE-6194
>                 URL: https://issues.apache.org/jira/browse/LUCENE-6194
>             Project: Lucene - Core
>          Issue Type: Bug
>          Components: modules/test-framework
>            Reporter: Michael McCandless
>            Priority: Minor
>
> I was playing with the patch on LUCENE-6066, and hit a test failure, but then 
> the "Reproduce with: " failed to reproduce, which threw me for a bit of a 
> loop, until I realized it was missing the -Dtests.iters=N.
> To see this, apply the patch on LUCENE-6066 to trunk and then run this:
> {noformat}
> ant test  -Dtestcase=TestPriorityQueue 
> -Dtests.method=testRemovalsAndInsertions* -Dtests.seed=2576D26D551748E5 
> -Dtests.slow=true 
> -Dtests.linedocsfile=/lucenedata/hudson.enwiki.random.lines.txt 
> -Dtests.locale=ru_RU -Dtests.timezone=Europe/Paris -Dtests.asserts=true 
> -Dtests.file.encoding=UTF-8 -Dtests.iters=10
> {noformat}
> (or just insert a random test failure into e.g. TestDemo or something).
> It should fail, but the "Reproduce with: ..." line does not include 
> -Dtests.iters=10.  I think it should?



--
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