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

Uwe Schindler commented on LUCENE-9158:
---------------------------------------

bq. its the simplest thing that can work, that's all. we can tweak the patch 
here to be more complicated 

The simplest thing unfortunately overrides the per module dirs and that makes 
debugging CI builds now impossible. You can't restore the state where the test 
work dir is per module, because you can only pass a static directory.

bq. I do think the default should be for ordinary users, jenkins can set a 
sysprop such as workDir instead? I think ant test should play nicely with the 
user's system out-of-box without a lot of special configuration.

Then we have to set this up different. We need 2 properties: One base dir for 
tests and one per module.

bq. I don't think running tests from 2 different checkouts at the same time is 
going to practically work. For example there are tests that bind to hardcoded 
network port numbers and will conflict with each other.

Works perfectly, I do this all the time. We fixed Solr long ago to use random 
port numbers for jetty (bind to 0). Of course I did not ran all tests, but 
looks correct.

> Fix tests to respect java.io.tmpdir by default
> ----------------------------------------------
>
>                 Key: LUCENE-9158
>                 URL: https://issues.apache.org/jira/browse/LUCENE-9158
>             Project: Lucene - Core
>          Issue Type: Improvement
>          Components: general/test
>            Reporter: Robert Muir
>            Priority: Major
>         Attachments: LUCENE-9158.patch
>
>
> Currently the tests destroy your SSDs by writing temp test data to 
> {{build/test/}} in your checkout.
> It would be better to respect {{java.io.tmpdir}} for where all the temporary 
> files get written. This way, if someone has configured their 
> system/environment variables appropriately, it won't destroy SSD.
> On linux it may work by default in many cases, as a lot of linux systems 
> mount {{/tmp}} on tmpfs by default.
> On windows users could have created a [RAM 
> drive|https://en.wikipedia.org/wiki/List_of_RAM_drive_software] such as 
> {{R:}} and have {{TMP}} environment variable pointed at it, we should respect 
> that.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

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

Reply via email to