> I found that passing -Ptests.timeoutSuite=500 doesn't have any effect > that I can see; it didn't interrupt the tests.
It's exactly what I said - without the exclamation mark, this is the default value of the timeout unless a class has an annotation specifying other value (which all of Lucene tests do because they inherit from LuceneTestCase). > I needed that trailing > exclamation mark for it to do the interrupt. Yep, it's an override that takes precedence over annotations. > majority. IMO LuceneTestCase shouldn't be declaring a default; it > should be done in a gradle build file instead. We could, sure. Or it could be shorter too. I don't have an opinion on this. > Then, configuration > for a build server (I'm thinking of Crave.io used by Solr PRs) can > specify like 10 minutes because otherwise an unlucky build hogs that > 96 core server for hours. Until then, I'll use an exclamation mark > for that server's config which isn't quite ideal but it's adequate. > +1.