RESOLVED - issue BATCH-1389: Thread safety in *PagingReader

Javadocs

spring-batch-infrastructure/src/main/java/org/springframework/batch/item/database/AbstractPagingItemReader.java

@@ -36,6 +36,7 @@ import org.springframework.util.ClassUtils;
  * </p>
  * 
  * @author Thomas Risberg
+ * @author Dave Syer
  * @since 2.0
  */
 public abstract class AbstractPagingItemReader<T> extends AbstractItemCountingItemStreamItemReader<T> implements

spring-batch-infrastructure/src/main/java/org/springframework/batch/item/database/IbatisPagingItemReader.java

@@ -18,6 +18,7 @@ package org.springframework.batch.item.database;
 
 import org.springframework.util.ClassUtils;
 import org.springframework.util.Assert;
+import org.springframework.batch.item.ExecutionContext;
 import org.springframework.orm.ibatis.SqlMapClientTemplate;
 
 import java.util.Map;
@@ -60,12 +61,14 @@ import com.ibatis.sqlmap.client.SqlMapClient;
  * </p>
  * 
  * <p>
- * The implementation is thread-safe, but remember to use
+ * The implementation is thread-safe in between calls to
+ * {@link #open(ExecutionContext)}, but remember to use
  * <code>saveState=false</code> if used in a multi-threaded client (no restart
  * available).
  * </p>
  * 
  * @author Thomas Risberg
+ * @author Dave Syer
  * @since 2.0
  */
 public class IbatisPagingItemReader<T> extends AbstractPagingItemReader<T> {

spring-batch-infrastructure/src/main/java/org/springframework/batch/item/database/JdbcPagingItemReader.java

@@ -28,6 +28,7 @@ import java.util.concurrent.CopyOnWriteArrayList;
 
 import javax.sql.DataSource;
 
+import org.springframework.batch.item.ExecutionContext;
 import org.springframework.beans.factory.InitializingBean;
 import org.springframework.jdbc.core.JdbcTemplate;
 import org.springframework.jdbc.core.RowCallbackHandler;
@@ -61,7 +62,8 @@ import org.springframework.util.ClassUtils;
  * </p>
  * 
  * <p>
- * The implementation is thread-safe, but remember to use
+ * The implementation is thread-safe in between calls to
+ * {@link #open(ExecutionContext)}, but remember to use
  * <code>saveState=false</code> if used in a multi-threaded client (no restart
  * available).
  * </p>

spring-batch-infrastructure/src/main/java/org/springframework/batch/item/database/JpaPagingItemReader.java

@@ -25,6 +25,7 @@ import javax.persistence.EntityManagerFactory;
 import javax.persistence.EntityTransaction;
 import javax.persistence.Query;
 
+import org.springframework.batch.item.ExecutionContext;
 import org.springframework.dao.DataAccessResourceFailureException;
 import org.springframework.util.Assert;
 import org.springframework.util.ClassUtils;
@@ -68,13 +69,15 @@ import org.springframework.util.ClassUtils;
  * </p>
  * 
  * <p>
- * The implementation is thread-safe, but remember to use
+ * The implementation is thread-safe in between calls to
+ * {@link #open(ExecutionContext)}, but remember to use
  * <code>saveState=false</code> if used in a multi-threaded client (no restart
  * available).
  * </p>
  * 
  * 
  * @author Thomas Risberg
+ * @author Dave Syer
  * @since 2.0
  */
 public class JpaPagingItemReader<T> extends AbstractPagingItemReader<T> {