[ 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