From 1287b2f53e53db964bf0dd2f793658546a6b17bb Mon Sep 17 00:00:00 2001 From: dsyer Date: Sun, 4 Apr 2010 16:31:39 +0000 Subject: [PATCH] Investigate test failure on append only set --- .../support/transaction/TransactionAwareProxyFactory.java | 4 ++-- .../transaction/ConcurrentTransactionAwareProxyTests.java | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/spring-batch-infrastructure/src/main/java/org/springframework/batch/support/transaction/TransactionAwareProxyFactory.java b/spring-batch-infrastructure/src/main/java/org/springframework/batch/support/transaction/TransactionAwareProxyFactory.java index 35da256b1..8901424e5 100644 --- a/spring-batch-infrastructure/src/main/java/org/springframework/batch/support/transaction/TransactionAwareProxyFactory.java +++ b/spring-batch-infrastructure/src/main/java/org/springframework/batch/support/transaction/TransactionAwareProxyFactory.java @@ -74,8 +74,8 @@ public class TransactionAwareProxyFactory { private TransactionAwareProxyFactory(T target, boolean appendOnly) { super(); - this.appendOnly = appendOnly; this.target = begin(target); + this.appendOnly = appendOnly; } /** @@ -123,7 +123,7 @@ public class TransactionAwareProxyFactory { */ @SuppressWarnings("unchecked") protected synchronized void commit(T copy, T target) { - // Unfortunately in Java 5 this method has to synchronized + // Unfortunately in Java 5 this method has to be synchronized // (works OK without in Java 6). if (target instanceof Collection) { if (!appendOnly) { diff --git a/spring-batch-infrastructure/src/test/java/org/springframework/batch/support/transaction/ConcurrentTransactionAwareProxyTests.java b/spring-batch-infrastructure/src/test/java/org/springframework/batch/support/transaction/ConcurrentTransactionAwareProxyTests.java index 68f9676f2..94278a9d9 100644 --- a/spring-batch-infrastructure/src/test/java/org/springframework/batch/support/transaction/ConcurrentTransactionAwareProxyTests.java +++ b/spring-batch-infrastructure/src/test/java/org/springframework/batch/support/transaction/ConcurrentTransactionAwareProxyTests.java @@ -50,7 +50,7 @@ public class ConcurrentTransactionAwareProxyTests { int outerMax = 10; - int innerMax = 10; + int innerMax = 15; private ExecutorService executor;