You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jmeter.apache.org by fs...@apache.org on 2019/09/27 16:52:50 UTC
[jmeter] branch master updated: Guard debug log messages
This is an automated email from the ASF dual-hosted git repository.
fschumacher pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/jmeter.git
The following commit(s) were added to refs/heads/master by this push:
new 0c8eb07 Guard debug log messages
0c8eb07 is described below
commit 0c8eb0785f9609d7db5b7494f9973ba2f7d74257
Author: Felix Schumacher <fe...@internetallee.de>
AuthorDate: Fri Sep 27 18:48:44 2019 +0200
Guard debug log messages
---
.../apache/jmeter/util/BeanShellTestElement.java | 24 ++++++++++++++++------
1 file changed, 18 insertions(+), 6 deletions(-)
diff --git a/src/core/src/main/java/org/apache/jmeter/util/BeanShellTestElement.java b/src/core/src/main/java/org/apache/jmeter/util/BeanShellTestElement.java
index 9a3a708..c681710 100644
--- a/src/core/src/main/java/org/apache/jmeter/util/BeanShellTestElement.java
+++ b/src/core/src/main/java/org/apache/jmeter/util/BeanShellTestElement.java
@@ -183,7 +183,9 @@ public abstract class BeanShellTestElement extends AbstractTestElement
try {
bshInterpreter.evalNoLog("threadStarted()"); // $NON-NLS-1$
} catch (JMeterException e) {
- log.debug(getClass().getName() + " : " + e.getLocalizedMessage()); // $NON-NLS-1$
+ if (log.isDebugEnabled()) {
+ log.debug(getClass().getName() + " : " + e.getLocalizedMessage()); // $NON-NLS-1$
+ }
}
}
@@ -195,7 +197,9 @@ public abstract class BeanShellTestElement extends AbstractTestElement
try {
bshInterpreter.evalNoLog("threadFinished()"); // $NON-NLS-1$
} catch (JMeterException e) {
- log.debug(getClass().getName() + " : " + e.getLocalizedMessage()); // $NON-NLS-1$
+ if (log.isDebugEnabled()) {
+ log.debug(getClass().getName() + " : " + e.getLocalizedMessage()); // $NON-NLS-1$
+ }
}
}
@@ -207,7 +211,9 @@ public abstract class BeanShellTestElement extends AbstractTestElement
try {
bshInterpreter.evalNoLog("testEnded()"); // $NON-NLS-1$
} catch (JMeterException e) {
- log.debug(getClass().getName() + " : " + e.getLocalizedMessage()); // $NON-NLS-1$
+ if (log.isDebugEnabled()) {
+ log.debug(getClass().getName() + " : " + e.getLocalizedMessage()); // $NON-NLS-1$
+ }
}
}
@@ -219,7 +225,9 @@ public abstract class BeanShellTestElement extends AbstractTestElement
try {
bshInterpreter.eval("testEnded(\"" + host + "\")");
} catch (JMeterException e) {
- log.debug(getClass().getName() + " : " + e.getLocalizedMessage()); // $NON-NLS-1$
+ if (log.isDebugEnabled()) {
+ log.debug(getClass().getName() + " : " + e.getLocalizedMessage()); // $NON-NLS-1$
+ }
}
}
@@ -231,7 +239,9 @@ public abstract class BeanShellTestElement extends AbstractTestElement
try {
bshInterpreter.evalNoLog("testStarted()"); // $NON-NLS-1$
} catch (JMeterException e) {
- log.debug(getClass().getName() + " : " + e.getLocalizedMessage()); // $NON-NLS-1$
+ if (log.isDebugEnabled()) {
+ log.debug(getClass().getName() + " : " + e.getLocalizedMessage()); // $NON-NLS-1$
+ }
}
}
@@ -243,7 +253,9 @@ public abstract class BeanShellTestElement extends AbstractTestElement
try {
bshInterpreter.eval("testStarted(\"" + host + "\")");
} catch (JMeterException e) {
- log.debug(getClass().getName() + " : " + e.getLocalizedMessage()); // $NON-NLS-1$
+ if (log.isDebugEnabled()) {
+ log.debug(getClass().getName() + " : " + e.getLocalizedMessage()); // $NON-NLS-1$
+ }
}
}