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

David Smiley commented on LUCENE-5645:
--------------------------------------

So it turns out this has to do with IntelliJ.  When IntelliJ configures itself 
off of a maven pom.xml, it apparently takes a close look at what system 
properties are being passed to the Surefire plugin, which runs tests.  If you 
tell IntelliJ to run a test class, it automatically passes along all System 
properties it saw that Surefire is configured to use (in addition to any you 
explicitly configure in a run configuration).  IntelliJ passes along the empty 
string if the System property is empty.  Apparently, an actual Maven Surefire 
invoked test skips over empty-value properties and doesn't set those.

Well that was interesting, and I'll file a bug with Jetbrains, but nonetheless 
I think my proposal in this issue remains sound -- don't try to parse 
tests.seed if it's "" because we already know it won't parse and because  
RandomizedTesting skips "" as well.

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