From 95cc70df4f08fe5a4f9cc5b41141bbfa64973927 Mon Sep 17 00:00:00 2001
From: dsyer <dsyer@2e8e966d-6415-0410-94b9-dc4e65435d03>
Date: Sat, 18 Sep 2010 08:05:17 +0000
Subject: [PATCH] BATCH-1629: add isDebugEnabled() to
 FaultTolerantChunkProcessor

---
 .../core/step/item/FaultTolerantChunkProcessor.java    | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)

diff --git a/spring-batch-core/src/main/java/org/springframework/batch/core/step/item/FaultTolerantChunkProcessor.java b/spring-batch-core/src/main/java/org/springframework/batch/core/step/item/FaultTolerantChunkProcessor.java
index 65ce0c459..74ac3aad1 100755
--- a/spring-batch-core/src/main/java/org/springframework/batch/core/step/item/FaultTolerantChunkProcessor.java
+++ b/spring-batch-core/src/main/java/org/springframework/batch/core/step/item/FaultTolerantChunkProcessor.java
@@ -381,7 +381,7 @@ public class FaultTolerantChunkProcessor<I, O> extends SimpleChunkProcessor<I, O
 								"Retry exhausted after last attempt in recovery path, but exception is not skippable.",
 								context.getLastThrowable());
 					}
-					
+
 					inputs.setBusy(true);
 					scan(contribution, inputs, outputs, chunkMonitor);
 					return null;
@@ -389,7 +389,9 @@ public class FaultTolerantChunkProcessor<I, O> extends SimpleChunkProcessor<I, O
 
 			};
 
-			logger.debug("Attempting to write: " + inputs);
+			if (logger.isDebugEnabled()) {
+				logger.debug("Attempting to write: " + inputs);
+			}
 			batchRetryTemplate.execute(retryCallback, recoveryCallback, new DefaultRetryState(inputs,
 					rollbackClassifier));
 
@@ -494,7 +496,9 @@ public class FaultTolerantChunkProcessor<I, O> extends SimpleChunkProcessor<I, O
 	private void scan(final StepContribution contribution, final Chunk<I> inputs, final Chunk<O> outputs,
 			ChunkMonitor chunkMonitor) throws Exception {
 
-		logger.debug("Scanning for failed item on write: " + inputs);
+		if (logger.isDebugEnabled()) {
+			logger.debug("Scanning for failed item on write: " + inputs);
+		}
 		if (outputs.isEmpty()) {
 			inputs.setBusy(false);
 			return;