OK I committed the fix for both active branches.
----- Uwe Schindler H.-H.-Meier-Allee 63, D-28213 Bremen <http://www.thetaphi.de/> http://www.thetaphi.de eMail: [email protected] From: Robert Muir [mailto:[email protected]] Sent: Wednesday, December 05, 2012 9:14 PM To: [email protected] Subject: Re: TestSqlEntityProcessorDelta failures on Policeman Jenkins +1 On Wed, Dec 5, 2012 at 3:08 PM, Uwe Schindler <[email protected]> wrote: OK, attached is a new patch with some refactoring (I renamed the property) and the missing modules added. I verified with grep the usage of tests.jvms in all build files, we are fine now. I also use the new property to force the number of JVMs for the clover runs! Uwe ----- Uwe Schindler H.-H.-Meier-Allee 63, D-28213 Bremen http://www.thetaphi.de <http://www.thetaphi.de/> eMail: [email protected] From: Robert Muir [mailto:[email protected]] Sent: Wednesday, December 05, 2012 8:57 PM To: [email protected] Subject: Re: TestSqlEntityProcessorDelta failures on Policeman Jenkins I guess i see the parallel tests as an optimization. To be safe we can keep the jvms set at 1 for these modules where its iffy. later after some investigation, we could re-enable them. but for now it would be great to get the number of sporatic failures down, and also great to have this option in the future in case we encounter a similar problem. On Wed, Dec 5, 2012 at 2:54 PM, Uwe Schindler <[email protected]> wrote: Thanks! But we never had a failure for the others, maybe remove the override for those modules? Definitely *all* Jenkins servers override the settings for JVM count (and also clover tests limit to 1). ----- Uwe Schindler H.-H.-Meier-Allee 63, D-28213 Bremen http://www.thetaphi.de <http://www.thetaphi.de/> eMail: [email protected] From: Robert Muir [mailto:[email protected]] Sent: Wednesday, December 05, 2012 8:51 PM To: [email protected] Subject: Re: TestSqlEntityProcessorDelta failures on Policeman Jenkins On Wed, Dec 5, 2012 at 2:48 PM, Uwe Schindler <[email protected]> wrote: I have a patch hat works, very simple! I will commit this for now. Are there any other modules that override the thread count? lucene/analysis/uima and lucene/benchmark also set it to 1. i don't know if its still strictly necessary, but we should do it for now to be safe. --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
