I agree with Gary that this test needs some work (or should not be part of the build: a proper performance test needs 5-10 seconds warmup, so these kind of tests end up taking too long to be run together with the functional JUnit tests).I don't think this test does what it is trying to do. (It won't detect new performance issues.)So I agree with Nick we don't need to treat this as a showstopper.RemkoPSFWIW, I cannot reproduce the issue on my PC at work.PS2Cut off lower half of this mail to prevent Apache mailer daemon from bouncing my message.
Sent from my iPhone
On 2013/06/04, at 9:42, Gary Gregory <email@example.com> wrote:Either there is a bug in the code, in the test, or the test should be excluded from running as part of the build, in which case, that should be documented in the test Javadoc. Something needs to be done IMO.
GaryOn Mon, Jun 3, 2013 at 8:32 PM, Nick Williams <firstname.lastname@example.org> wrote:
The three failing tests are in SimplePerfTest, and the error on all of them is that the timer was exceeded. This should obviously be looked at either way, but it may be okay to release a beta with these tests failing IF they are truly only failing because a task took to long, and not because something is broken.That's my opinion, of course.NickOn Jun 3, 2013, at 6:58 PM, Gary Gregory wrote:-1: I see unit test failures: https://people.apache.org/~rgoers/log4j2/log4j-core/surefire-report.htmlGary
On Sun, Jun 2, 2013 at 10:41 AM, Ralph Goers <email@example.com> wrote:
This is a vote to release Log4j 2.0-beta7, the ninth release of Log4j 2.0.