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

Dawid Weiss commented on LUCENE-5645:
-------------------------------------

> Why is randomized testing lenient?

It parses an empty value of the property because it's more convenient sometimes 
to pass it from ant as empty (rather then not defining it at all).

The start = (prop == null ? 0 : prop.hashCode()) seems like a neat trick for 
seeding the hash random though.

> StringHelper should check for empty string of "tests.seed" system property
> --------------------------------------------------------------------------
>
>                 Key: LUCENE-5645
>                 URL: https://issues.apache.org/jira/browse/LUCENE-5645
>             Project: Lucene - Core
>          Issue Type: Bug
>    Affects Versions: 4.8, 5.0
>            Reporter: David Smiley
>            Assignee: David Smiley
>            Priority: Minor
>             Fix For: 4.8.1, 5.0
>
>         Attachments: LUCENE-5645_StringHelper_empty_tests_seed_condition.patch
>
>
> As of LUCENE-5604 (committed to v4.8), StringHelper will initialize 
> GOOD_FAST_HASH_SEED based on the system property "tests.seed" if it is set.  
> Unfortunately it doesn't do an empty-string check, and it's common at least 
> in my setup that copies Lucene's maven pom.xml that the string will be empty 
> unless I set it on the command line.  FWIW Randomized Testing does do an 
> empty-string check.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org

Reply via email to