+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**** > > 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**** > > 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] >
