Hi Felix, Can’t this have an impact on performances leading to a lot of error logging ?
Regards On Thursday, February 15, 2018, <fschumac...@apache.org> wrote: > Author: fschumacher > Date: Thu Feb 15 20:09:16 2018 > New Revision: 1824355 > > URL: http://svn.apache.org/viewvc?rev=1824355&view=rev > Log: > JUnitSampler logs exceptions except assertion-failures from test cases as > warnings. > > Contributed by Davide Angelocola (davide.angelocola at fisglobal.com) > Closes #376 on github > > Modified: > jmeter/trunk/src/junit/org/apache/jmeter/protocol/java/ > sampler/JUnitSampler.java > jmeter/trunk/xdocs/changes.xml > > Modified: jmeter/trunk/src/junit/org/apache/jmeter/protocol/java/ > sampler/JUnitSampler.java > URL: http://svn.apache.org/viewvc/jmeter/trunk/src/junit/org/ > apache/jmeter/protocol/java/sampler/JUnitSampler.java?rev= > 1824355&r1=1824354&r2=1824355&view=diff > ============================================================ > ================== > --- > jmeter/trunk/src/junit/org/apache/jmeter/protocol/java/sampler/JUnitSampler.java > (original) > +++ > jmeter/trunk/src/junit/org/apache/jmeter/protocol/java/sampler/JUnitSampler.java > Thu Feb 15 20:09:16 2018 > @@ -410,8 +410,10 @@ public class JUnitSampler extends Abstra > tr.addFailure(theClazz, afe); > } else if (cause != null) { > tr.addError(theClazz, cause); > + log.warn("caught exception", e); > } else { > tr.addError(theClazz, e); > + log.warn("caught exception", e); > } > } catch (IllegalAccessException | IllegalArgumentException e) > { > tr.addError(theClazz, e); > > Modified: jmeter/trunk/xdocs/changes.xml > URL: http://svn.apache.org/viewvc/jmeter/trunk/xdocs/changes. > xml?rev=1824355&r1=1824354&r2=1824355&view=diff > ============================================================ > ================== > --- jmeter/trunk/xdocs/changes.xml [utf-8] (original) > +++ jmeter/trunk/xdocs/changes.xml [utf-8] Thu Feb 15 20:09:16 2018 > @@ -86,6 +86,7 @@ Summary > > <h3>Other samplers</h3> > <ul> > + <li><pr>376</pr>JUnitSampler logs exceptions except assertion-failures > from test cases as warnings. Contributed by Davide Angelocola > (davide.angelocola at fisglobal.com)</li> > </ul> > > <h3>Controllers</h3> > @@ -174,6 +175,7 @@ Summary > <ul> > <li>Graham Russell (graham at ham1.co.uk)</li> > <li>orimarko at gmail.com</li> > + <li>Davide Angelocola (davide.angelocola at fisglobal.com)</li> > </ul> > <p>We also thank bug reporters who helped us improve JMeter.</p> > <p> > > > -- Cordialement. Philippe Mouawad.
