Commit 681a94b0 authored by wycm's avatar wycm Committed by Stephane Nicoll

Optimize debug level logs

See gh-18604
parent 91f4300f
......@@ -174,7 +174,9 @@ public class JobLauncherCommandLineRunner implements CommandLineRunner, Ordered,
if (StringUtils.hasText(this.jobNames)) {
String[] jobsToRun = this.jobNames.split(",");
if (!PatternMatchUtils.simpleMatch(jobsToRun, job.getName())) {
logger.debug("Skipped job: " + job.getName());
if (logger.isDebugEnabled()) {
logger.debug("Skipped job: " + job.getName());
}
continue;
}
}
......@@ -194,7 +196,9 @@ public class JobLauncherCommandLineRunner implements CommandLineRunner, Ordered,
execute(job, jobParameters);
}
catch (NoSuchJobException ex) {
logger.debug("No job found in registry for job name: " + jobName);
if (logger.isDebugEnabled()) {
logger.debug("No job found in registry for job name: " + jobName);
}
}
}
}
......
......@@ -63,7 +63,7 @@ class ArtemisEmbeddedConfigurationFactory {
TransportConfiguration transportConfiguration = new TransportConfiguration(InVMAcceptorFactory.class.getName(),
this.properties.generateTransportParameters());
configuration.getAcceptorConfigurations().add(transportConfiguration);
if (this.properties.isDefaultClusterPassword()) {
if (this.properties.isDefaultClusterPassword() && logger.isDebugEnabled()) {
logger.debug("Using default Artemis cluster password: " + this.properties.getClusterPassword());
}
configuration.setClusterPassword(this.properties.getClusterPassword());
......
......@@ -68,7 +68,9 @@ class Connection {
this.inputStream = new ConnectionInputStream(inputStream);
this.outputStream = new ConnectionOutputStream(outputStream);
this.header = this.inputStream.readHeader();
logger.debug("Established livereload connection [" + this.header + "]");
if (logger.isDebugEnabled()) {
logger.debug("Established livereload connection [" + this.header + "]");
}
}
/**
......@@ -107,7 +109,9 @@ class Connection {
throw new ConnectionClosedException();
}
else if (frame.getType() == Frame.Type.TEXT) {
logger.debug("Received LiveReload text frame " + frame);
if (logger.isDebugEnabled()) {
logger.debug("Received LiveReload text frame " + frame);
}
}
else {
throw new IOException("Unexpected Frame Type " + frame.getType());
......
......@@ -109,7 +109,9 @@ public class LiveReloadServer {
public int start() throws IOException {
synchronized (this.monitor) {
Assert.state(!isStarted(), "Server already started");
logger.debug("Starting live reload server on port " + this.port);
if (logger.isDebugEnabled()) {
logger.debug("Starting live reload server on port " + this.port);
}
this.serverSocket = new ServerSocket(this.port);
int localPort = this.serverSocket.getLocalPort();
this.listenThread = this.threadFactory.newThread(this::acceptConnections);
......
......@@ -113,7 +113,9 @@ final class FailureAnalyzers implements SpringBootExceptionReporter {
}
}
catch (Throwable ex) {
logger.debug("FailureAnalyzer " + analyzer + " failed", ex);
if (logger.isDebugEnabled()) {
logger.debug("FailureAnalyzer " + analyzer + " failed", ex);
}
}
}
return null;
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment