diff --git a/build-spring-batch/build.xml b/build-spring-batch/build.xml
deleted file mode 100644
index 5006155d5..000000000
--- a/build-spring-batch/build.xml
+++ /dev/null
@@ -1,26 +0,0 @@
-
-
* Enable Spring Batch features and provide a base configuration for setting up batch jobs in an @Configuration
- * class, roughly equivalent to using the {@code
- *
+ * alphabetically greater.
+ *
*
* Severity is defined by the exit code:
*
@@ -117,7 +117,7 @@ public class ExitStatus implements Serializable, Comparable
- * Others have severity 7, so custom exit codes always win.
+ * Others have severity 7, so custom exit codes always win.
*
* If the input is null just return this.
*
diff --git a/spring-batch-core/src/main/java/org/springframework/batch/core/JobExecution.java b/spring-batch-core/src/main/java/org/springframework/batch/core/JobExecution.java
index 45be8fc33..71b352294 100644
--- a/spring-batch-core/src/main/java/org/springframework/batch/core/JobExecution.java
+++ b/spring-batch-core/src/main/java/org/springframework/batch/core/JobExecution.java
@@ -16,6 +16,8 @@
package org.springframework.batch.core;
+import org.springframework.batch.item.ExecutionContext;
+
import java.io.IOException;
import java.io.ObjectInputStream;
import java.util.ArrayList;
@@ -28,8 +30,6 @@ import java.util.Set;
import java.util.concurrent.CopyOnWriteArrayList;
import java.util.concurrent.CopyOnWriteArraySet;
-import org.springframework.batch.item.ExecutionContext;
-
/**
* Batch domain object representing the execution of a job.
*
@@ -333,7 +333,7 @@ public class JobExecution extends Entity {
* Return all failure causing exceptions for this JobExecution, including
* step executions.
*
- * @return List
- *
+ * finished.
+ *
* Using the identifying flag indicates if the parameter will be used
* in the identification of a JobInstance. That flag defaults to true.
*
diff --git a/spring-batch-core/src/main/java/org/springframework/batch/core/Step.java b/spring-batch-core/src/main/java/org/springframework/batch/core/Step.java
index df0d61096..c5e83e4b8 100644
--- a/spring-batch-core/src/main/java/org/springframework/batch/core/Step.java
+++ b/spring-batch-core/src/main/java/org/springframework/batch/core/Step.java
@@ -42,7 +42,7 @@ public interface Step {
/**
* Process the step and assign progress and status meta information to the {@link StepExecution} provided. The
* {@link Step} is responsible for setting the meta information and also saving it if required by the
- * implementation.
+ * implementation.
*
* It is not safe to re-use an instance of {@link Step} to process multiple concurrent executions.
*
diff --git a/spring-batch-core/src/main/java/org/springframework/batch/core/annotation/AfterChunk.java b/spring-batch-core/src/main/java/org/springframework/batch/core/annotation/AfterChunk.java
index 2a0366f59..003a8b21e 100644
--- a/spring-batch-core/src/main/java/org/springframework/batch/core/annotation/AfterChunk.java
+++ b/spring-batch-core/src/main/java/org/springframework/batch/core/annotation/AfterChunk.java
@@ -15,17 +15,17 @@
*/
package org.springframework.batch.core.annotation;
+import org.springframework.batch.core.ChunkListener;
+import org.springframework.batch.core.scope.context.ChunkContext;
+
import java.lang.annotation.ElementType;
import java.lang.annotation.Retention;
import java.lang.annotation.RetentionPolicy;
import java.lang.annotation.Target;
-import org.springframework.batch.core.ChunkListener;
-import org.springframework.batch.core.scope.context.ChunkContext;
-
/**
* Marks a method to be called after a chunk is executed.
- *
+ *
* Expected signature: void afterChunk(ChunkContext context)
*
* @author Lucas Ward
diff --git a/spring-batch-core/src/main/java/org/springframework/batch/core/annotation/AfterChunkError.java b/spring-batch-core/src/main/java/org/springframework/batch/core/annotation/AfterChunkError.java
index eba8a66a2..a8ef5fce5 100644
--- a/spring-batch-core/src/main/java/org/springframework/batch/core/annotation/AfterChunkError.java
+++ b/spring-batch-core/src/main/java/org/springframework/batch/core/annotation/AfterChunkError.java
@@ -15,18 +15,18 @@
*/
package org.springframework.batch.core.annotation;
+import org.springframework.batch.core.ChunkListener;
+import org.springframework.batch.core.scope.context.ChunkContext;
+
import java.lang.annotation.ElementType;
import java.lang.annotation.Retention;
import java.lang.annotation.RetentionPolicy;
import java.lang.annotation.Target;
-import org.springframework.batch.core.ChunkListener;
-import org.springframework.batch.core.scope.context.ChunkContext;
-
/**
* Marks a method to be called after a has failed and been
* marked for rollback.
- *
+ *
* Expected signature: void afterFailedChunk(ChunkContext context)
*
* @author Michael Minella
diff --git a/spring-batch-core/src/main/java/org/springframework/batch/core/configuration/annotation/EnableBatchProcessing.java b/spring-batch-core/src/main/java/org/springframework/batch/core/configuration/annotation/EnableBatchProcessing.java
index 662197c0c..cedf8cc83 100644
--- a/spring-batch-core/src/main/java/org/springframework/batch/core/configuration/annotation/EnableBatchProcessing.java
+++ b/spring-batch-core/src/main/java/org/springframework/batch/core/configuration/annotation/EnableBatchProcessing.java
@@ -33,7 +33,7 @@ import java.lang.annotation.Target;
/**
*
* @Configuration
@@ -134,8 +134,6 @@ import java.lang.annotation.Target;
* especially if a {@link BatchConfigurer} is provided, because cyclic configuration dependencies are otherwise likely
* to develop.
*
- *
* For reference, the first example above can be compared to the following Spring XML configuration: * diff --git a/spring-batch-core/src/main/java/org/springframework/batch/core/configuration/annotation/StepScope.java b/spring-batch-core/src/main/java/org/springframework/batch/core/configuration/annotation/StepScope.java index b95c4c129..19e66489e 100644 --- a/spring-batch-core/src/main/java/org/springframework/batch/core/configuration/annotation/StepScope.java +++ b/spring-batch-core/src/main/java/org/springframework/batch/core/configuration/annotation/StepScope.java @@ -15,13 +15,13 @@ */ package org.springframework.batch.core.configuration.annotation; +import org.springframework.context.annotation.Scope; +import org.springframework.context.annotation.ScopedProxyMode; + import java.lang.annotation.Documented; import java.lang.annotation.Retention; import java.lang.annotation.RetentionPolicy; -import org.springframework.context.annotation.Scope; -import org.springframework.context.annotation.ScopedProxyMode; - /** *
* Convenient annotation for step scoped beans that defaults the proxy mode, so that it doesn't have to be specified @@ -42,7 +42,7 @@ import org.springframework.context.annotation.ScopedProxyMode; * * @author Dave Syer * - * @Since 2.2 + * @since 2.2 * */ @Scope(value = "step", proxyMode = ScopedProxyMode.TARGET_CLASS) diff --git a/spring-batch-core/src/main/java/org/springframework/batch/core/configuration/support/DefaultJobLoader.java b/spring-batch-core/src/main/java/org/springframework/batch/core/configuration/support/DefaultJobLoader.java index 260618e28..bfec62a1b 100644 --- a/spring-batch-core/src/main/java/org/springframework/batch/core/configuration/support/DefaultJobLoader.java +++ b/spring-batch-core/src/main/java/org/springframework/batch/core/configuration/support/DefaultJobLoader.java @@ -15,12 +15,6 @@ */ package org.springframework.batch.core.configuration.support; -import java.util.ArrayList; -import java.util.Collection; -import java.util.HashSet; -import java.util.Map; -import java.util.concurrent.ConcurrentHashMap; - import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory; import org.springframework.batch.core.Job; @@ -36,6 +30,12 @@ import org.springframework.context.ApplicationContext; import org.springframework.context.ConfigurableApplicationContext; import org.springframework.util.Assert; +import java.util.ArrayList; +import java.util.Collection; +import java.util.HashSet; +import java.util.Map; +import java.util.concurrent.ConcurrentHashMap; + /** * Default implementation of {@link JobLoader}. Uses a {@link JobRegistry} to * manage a population of loaded jobs and clears them up when asked. An optional @@ -208,7 +208,7 @@ public class DefaultJobLoader implements JobLoader, InitializingBean { /** * Returns all the {@link Step} instances defined by the specified {@link StepLocator}. - *
+ *This class is considered a quick work around and needs to be refactored / removed.
* *The in addition to making this class package private, the following methods were modified to be protected:
- ** Basic launcher for starting jobs from the command line. In general, it is @@ -101,11 +101,12 @@ import org.springframework.util.StringUtils; *
* *
- * jobPath jobIdentifier (jobParameters)*
+ * jobPath <options> jobIdentifier (jobParameters)*
*
*
* * The command line options are as follows + *
*key=value pairs.
*
* If the -next option is used the parameters on the command line
@@ -135,18 +135,18 @@ import org.springframework.util.StringUtils;
* {@link JobParameters} using a {@link JobParametersConverter} from the
* application context (if there is one, or a
* {@link DefaultJobParametersConverter} otherwise). Below is an example
- * arguments list: "
+ * arguments list: "
*
* java org.springframework.batch.core.launch.support.CommandLineJobRunner testJob.xml
* testJob schedule.date=2008/01/24 vendor.id=3902483920
- *
+ *
*
* Once arguments have been successfully parsed, autowiring will be used to set
- * various dependencies. The {@JobLauncher} for example, will be
+ * various dependencies. The {@link JobLauncher} for example, will be
* loaded this way. If none is contained in the bean factory (it searches by
* type) then a {@link BeanDefinitionStoreException} will be thrown. The same
* exception will also be thrown if there is more than one present. Assuming the
@@ -502,11 +502,11 @@ public class CommandLineJobRunner {
* such contexts. No exception are thrown from this method, rather
* exceptions are logged and an integer returned through the exit status in
* a {@link JvmSystemExiter} (which can be overridden by defining one in the
- * Spring context).
+ * Spring context).
* Parameters can be provided in the form key=value, and will be converted
* using the injected {@link JobParametersConverter}.
*
- * @param args
+ * @param args *
* The options (-restart, -next) can occur anywhere in the
* command line.
*
* Repository responsible for persistence of batch meta-data entities. @@ -100,7 +100,7 @@ public interface JobRepository { * support the higher isolation levels). *
* - * @param jobName the name of the job that is to be executed + * @param jobName the name of the job that is to be executed * * @param jobParameters the runtime parameters for the job * diff --git a/spring-batch-core/src/main/java/org/springframework/batch/core/repository/dao/JdbcStepExecutionDao.java b/spring-batch-core/src/main/java/org/springframework/batch/core/repository/dao/JdbcStepExecutionDao.java index 42c9a347e..2ffcf03ab 100644 --- a/spring-batch-core/src/main/java/org/springframework/batch/core/repository/dao/JdbcStepExecutionDao.java +++ b/spring-batch-core/src/main/java/org/springframework/batch/core/repository/dao/JdbcStepExecutionDao.java @@ -16,17 +16,6 @@ package org.springframework.batch.core.repository.dao; -import java.sql.PreparedStatement; -import java.sql.ResultSet; -import java.sql.SQLException; -import java.sql.Timestamp; -import java.sql.Types; -import java.util.ArrayList; -import java.util.Arrays; -import java.util.Collection; -import java.util.Iterator; -import java.util.List; - import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory; import org.springframework.batch.core.BatchStatus; @@ -40,18 +29,29 @@ import org.springframework.jdbc.core.simple.ParameterizedRowMapper; import org.springframework.jdbc.support.incrementer.DataFieldMaxValueIncrementer; import org.springframework.util.Assert; +import java.sql.PreparedStatement; +import java.sql.ResultSet; +import java.sql.SQLException; +import java.sql.Timestamp; +import java.sql.Types; +import java.util.ArrayList; +import java.util.Arrays; +import java.util.Collection; +import java.util.Iterator; +import java.util.List; + /** - * JDBC implementation of {@link StepExecutionDao}.Exception
@@ -199,7 +199,7 @@ public class FaultTolerantStepFactoryBeanskipCount<0 to probe for exception types that are skippable,
+ * {@code skipCount<0} to probe for exception types that are skippable,
* so implementations should be able to handle gracefully the case where
- * skipCount<0. Implementations should avoid throwing any
+ * {@code skipCount<0}. Implementations should avoid throwing any
* undeclared exceptions.
*
* @param t exception encountered while reading
diff --git a/spring-batch-core/src/main/java/org/springframework/batch/core/step/tasklet/SimpleSystemProcessExitCodeMapper.java b/spring-batch-core/src/main/java/org/springframework/batch/core/step/tasklet/SimpleSystemProcessExitCodeMapper.java
index ba27551f9..55c7b4ab7 100644
--- a/spring-batch-core/src/main/java/org/springframework/batch/core/step/tasklet/SimpleSystemProcessExitCodeMapper.java
+++ b/spring-batch-core/src/main/java/org/springframework/batch/core/step/tasklet/SimpleSystemProcessExitCodeMapper.java
@@ -21,8 +21,8 @@ import org.springframework.batch.core.ExitStatus;
/**
* Simple {@link SystemProcessExitCodeMapper} implementation that performs following mapping:
*
- * 0 -> ExitStatus.FINISHED
- * else -> ExitStatus.FAILED
+ * 0 -> ExitStatus.FINISHED
+ * else -> ExitStatus.FAILED
*
* @author Robert Kasanicky
*/
diff --git a/spring-batch-core/src/main/java/org/springframework/batch/core/step/tasklet/TaskletStep.java b/spring-batch-core/src/main/java/org/springframework/batch/core/step/tasklet/TaskletStep.java
index be8e58180..096570916 100644
--- a/spring-batch-core/src/main/java/org/springframework/batch/core/step/tasklet/TaskletStep.java
+++ b/spring-batch-core/src/main/java/org/springframework/batch/core/step/tasklet/TaskletStep.java
@@ -15,8 +15,6 @@
*/
package org.springframework.batch.core.step.tasklet;
-import java.util.concurrent.Semaphore;
-
import org.apache.commons.logging.Log;
import org.apache.commons.logging.LogFactory;
import org.springframework.batch.core.BatchStatus;
@@ -53,13 +51,15 @@ import org.springframework.transaction.support.TransactionSynchronizationManager
import org.springframework.transaction.support.TransactionTemplate;
import org.springframework.util.Assert;
+import java.util.concurrent.Semaphore;
+
/**
* Simple implementation of executing the step as a call to a {@link Tasklet},
* possibly repeated, and each call surrounded by a transaction. The structure
* is therefore that of a loop with transaction boundary inside the loop. The
* loop is controlled by the step operations (
- * {@link #setStepOperations(RepeatOperations)}).null when all input data is exhausted.null when all input data is exhausted.@@ -52,7 +50,7 @@ import com.ibatis.sqlmap.client.SqlMapSession; * to current position. Some standard query parameters are provided by the * reader and the SQL in the named query must use some or all of these parameters * (depending on the SQL variant) to construct a result set of the required - * size. The parameters are: + * size. The parameters are:
*_page: the page number to be read (starting at 0)_pagesize: the size of the pages, i.e. the number of rows to
@@ -60,6 +58,7 @@ import com.ibatis.sqlmap.client.SqlMapSession;
* _skiprows: the product of _page and
* _pagesize* Failure to write the correct platform-specific SQL often results in an * infinite loop in the reader because it keeps asking for the next page and * gets the same result set over and over. diff --git a/spring-batch-infrastructure/src/main/java/org/springframework/batch/item/database/JdbcBatchItemWriter.java b/spring-batch-infrastructure/src/main/java/org/springframework/batch/item/database/JdbcBatchItemWriter.java index 95f0252a1..0f1f33390 100644 --- a/spring-batch-infrastructure/src/main/java/org/springframework/batch/item/database/JdbcBatchItemWriter.java +++ b/spring-batch-infrastructure/src/main/java/org/springframework/batch/item/database/JdbcBatchItemWriter.java @@ -15,14 +15,6 @@ */ package org.springframework.batch.item.database; -import java.sql.PreparedStatement; -import java.sql.SQLException; -import java.util.ArrayList; -import java.util.List; -import java.util.Map; - -import javax.sql.DataSource; - import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory; import org.springframework.batch.item.ItemWriter; @@ -36,6 +28,13 @@ import org.springframework.jdbc.core.namedparam.NamedParameterJdbcTemplate; import org.springframework.jdbc.core.namedparam.SqlParameterSource; import org.springframework.util.Assert; +import javax.sql.DataSource; +import java.sql.PreparedStatement; +import java.sql.SQLException; +import java.util.ArrayList; +import java.util.List; +import java.util.Map; + /** *
{@link ItemWriter} that uses the batching features from
* {@link NamedParameterJdbcTemplate} to execute a batch of statements for all items
@@ -47,9 +46,9 @@ import org.springframework.util.Assert;
* named parameter support then you should provide a {@link ItemSqlParameterSourceProvider},
* otherwise you should provide a {@link ItemPreparedStatementSetter}.
* This callback would be responsible for mapping the item to the parameters needed to
- * execute the SQL statement.
+ * execute the SQL statement.
*
- * It is expected that {@link #write(List)} is called inside a transaction.
+ * It is expected that {@link #write(List)} is called inside a transaction.
*
* The writer is thread-safe after its properties are set (normal singleton
* behavior), so it can be used to write in multiple concurrent transactions.
diff --git a/spring-batch-infrastructure/src/main/java/org/springframework/batch/item/database/JdbcParameterUtils.java b/spring-batch-infrastructure/src/main/java/org/springframework/batch/item/database/JdbcParameterUtils.java
index 51553b859..7c6aafe13 100644
--- a/spring-batch-infrastructure/src/main/java/org/springframework/batch/item/database/JdbcParameterUtils.java
+++ b/spring-batch-infrastructure/src/main/java/org/springframework/batch/item/database/JdbcParameterUtils.java
@@ -36,7 +36,7 @@ public class JdbcParameterUtils {
* sql. The character placeholder is not counted if it appears
* within a literal, that is, surrounded by single or double quotes. This method will
* count traditional placeholders in the form of a question mark ('?') as well as
- * named parameters indicated with a leading ':' or '&'.
+ * named parameters indicated with a leading ':' or '&'.
*
* The code for this method is taken from an early version of the
* {@link org.springframework.jdbc.core.namedparam.NamedParameterUtils}
diff --git a/spring-batch-infrastructure/src/main/java/org/springframework/batch/item/database/JpaItemWriter.java b/spring-batch-infrastructure/src/main/java/org/springframework/batch/item/database/JpaItemWriter.java
index 2f81aa482..d58f129fc 100644
--- a/spring-batch-infrastructure/src/main/java/org/springframework/batch/item/database/JpaItemWriter.java
+++ b/spring-batch-infrastructure/src/main/java/org/springframework/batch/item/database/JpaItemWriter.java
@@ -16,11 +16,6 @@
package org.springframework.batch.item.database;
-import java.util.List;
-
-import javax.persistence.EntityManager;
-import javax.persistence.EntityManagerFactory;
-
import org.apache.commons.logging.Log;
import org.apache.commons.logging.LogFactory;
import org.springframework.batch.item.ItemWriter;
@@ -29,12 +24,16 @@ import org.springframework.dao.DataAccessResourceFailureException;
import org.springframework.orm.jpa.EntityManagerFactoryUtils;
import org.springframework.util.Assert;
+import javax.persistence.EntityManager;
+import javax.persistence.EntityManagerFactory;
+import java.util.List;
+
/**
* {@link org.springframework.batch.item.ItemWriter} that is using a JPA
* EntityManagerFactory to merge any Entities that aren't part of the
* persistence context.
*
- * It is required that {@link #write(List)} is called inside a transaction.
+ * It is required that {@link #write(List)} is called inside a transaction.
*
* The reader must be configured with an
* {@link javax.persistence.EntityManagerFactory} that is capable of
diff --git a/spring-batch-infrastructure/src/main/java/org/springframework/batch/item/database/support/AbstractSqlPagingQueryProvider.java b/spring-batch-infrastructure/src/main/java/org/springframework/batch/item/database/support/AbstractSqlPagingQueryProvider.java
index eb090fd5a..c195cd993 100644
--- a/spring-batch-infrastructure/src/main/java/org/springframework/batch/item/database/support/AbstractSqlPagingQueryProvider.java
+++ b/spring-batch-infrastructure/src/main/java/org/springframework/batch/item/database/support/AbstractSqlPagingQueryProvider.java
@@ -16,13 +16,6 @@
package org.springframework.batch.item.database.support;
-import java.util.ArrayList;
-import java.util.LinkedHashMap;
-import java.util.List;
-import java.util.Map;
-
-import javax.sql.DataSource;
-
import org.springframework.batch.item.database.JdbcParameterUtils;
import org.springframework.batch.item.database.Order;
import org.springframework.batch.item.database.PagingQueryProvider;
@@ -30,6 +23,12 @@ import org.springframework.dao.InvalidDataAccessApiUsageException;
import org.springframework.util.Assert;
import org.springframework.util.StringUtils;
+import javax.sql.DataSource;
+import java.util.ArrayList;
+import java.util.LinkedHashMap;
+import java.util.List;
+import java.util.Map;
+
/**
* Abstract SQL Paging Query Provider to serve as a base class for all provided
* SQL paging query providers.
@@ -147,7 +146,7 @@ public abstract class AbstractSqlPagingQueryProvider implements PagingQueryProvi
}
/**
- * A Map
Implementation of the {@link ItemPreparedStatementSetter} interface that assumes all * keys are contained within a {@link Map} with the column name as the key. It assumes nothing * about ordering, and assumes that the order the entry set can be iterated over is the same as * the PreparedStatement should be set.
diff --git a/spring-batch-infrastructure/src/main/java/org/springframework/batch/item/file/FlatFileItemWriter.java b/spring-batch-infrastructure/src/main/java/org/springframework/batch/item/file/FlatFileItemWriter.java index a0eca4936..74fabc421 100644 --- a/spring-batch-infrastructure/src/main/java/org/springframework/batch/item/file/FlatFileItemWriter.java +++ b/spring-batch-infrastructure/src/main/java/org/springframework/batch/item/file/FlatFileItemWriter.java @@ -16,16 +16,6 @@ package org.springframework.batch.item.file; -import java.io.BufferedWriter; -import java.io.File; -import java.io.FileOutputStream; -import java.io.IOException; -import java.io.Writer; -import java.nio.channels.Channels; -import java.nio.channels.FileChannel; -import java.nio.charset.UnsupportedCharsetException; -import java.util.List; - import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory; import org.springframework.batch.item.ExecutionContext; @@ -42,12 +32,22 @@ import org.springframework.core.io.Resource; import org.springframework.util.Assert; import org.springframework.util.ClassUtils; +import java.io.BufferedWriter; +import java.io.File; +import java.io.FileOutputStream; +import java.io.IOException; +import java.io.Writer; +import java.nio.channels.Channels; +import java.nio.channels.FileChannel; +import java.nio.charset.UnsupportedCharsetException; +import java.util.List; + /** * This class is an item writer that writes data to a file or stream. The writer * also provides restart. The location of the output file is defined by a - * {@link Resource} and must represent a writable file.mydir/*.txt, which can then be converted by Spring
* to an array of Resources by the ApplicationContext.
*
- * index'.
*
- * @returns null if the field value is null.
+ * @return null if the field value is null.
*/
protected String readAndTrim(int index) {
String value = tokens[index];
diff --git a/spring-batch-infrastructure/src/main/java/org/springframework/batch/item/file/transform/FormatterLineAggregator.java b/spring-batch-infrastructure/src/main/java/org/springframework/batch/item/file/transform/FormatterLineAggregator.java
index 063df94dc..8b6f41fa2 100644
--- a/spring-batch-infrastructure/src/main/java/org/springframework/batch/item/file/transform/FormatterLineAggregator.java
+++ b/spring-batch-infrastructure/src/main/java/org/springframework/batch/item/file/transform/FormatterLineAggregator.java
@@ -23,7 +23,7 @@ import org.springframework.util.Assert;
/**
* A {@link LineAggregator} implementation which produces a String by
- * aggregating the provided item via the {@link Formatter} syntax.
+ * aggregating the provided item via the {@link Formatter} syntax.
- * Examples:
- * '1, 15, 25, 38, 55-60' is equal to '1-14, 15-24, 25-37, 38-54, 55-60'
+ * Examples:
+ * '1, 15, 25, 38, 55-60' is equal to '1-14, 15-24, 25-37, 38-54, 55-60'
* '36, 14, 1-10, 15, 49-57' is equal to '36-48, 14-14, 1-10, 15-35, 49-57'
*
* Property editor also allows to validate whether ranges are disjoint. Validation
diff --git a/spring-batch-infrastructure/src/main/java/org/springframework/batch/item/file/transform/RecursiveCollectionLineAggregator.java b/spring-batch-infrastructure/src/main/java/org/springframework/batch/item/file/transform/RecursiveCollectionLineAggregator.java
index 4b75752e9..4ab2beab5 100644
--- a/spring-batch-infrastructure/src/main/java/org/springframework/batch/item/file/transform/RecursiveCollectionLineAggregator.java
+++ b/spring-batch-infrastructure/src/main/java/org/springframework/batch/item/file/transform/RecursiveCollectionLineAggregator.java
@@ -35,7 +35,7 @@ public class RecursiveCollectionLineAggregator
*
* @param delegate the line aggregator to set. Defaults to a pass through.
*/
diff --git a/spring-batch-infrastructure/src/main/java/org/springframework/batch/item/jms/JmsItemReader.java b/spring-batch-infrastructure/src/main/java/org/springframework/batch/item/jms/JmsItemReader.java
index ced399ff4..6a29fa7f1 100644
--- a/spring-batch-infrastructure/src/main/java/org/springframework/batch/item/jms/JmsItemReader.java
+++ b/spring-batch-infrastructure/src/main/java/org/springframework/batch/item/jms/JmsItemReader.java
@@ -16,8 +16,6 @@
package org.springframework.batch.item.jms;
-import javax.jms.Message;
-
import org.apache.commons.logging.Log;
import org.apache.commons.logging.LogFactory;
import org.springframework.batch.item.ItemReader;
@@ -26,11 +24,13 @@ import org.springframework.jms.core.JmsOperations;
import org.springframework.jms.core.JmsTemplate;
import org.springframework.util.Assert;
+import javax.jms.Message;
+
/**
* An {@link ItemReader} for JMS using a {@link JmsTemplate}. The template
* should have a default destination, which will be used to provide items in
- * {@link #read()}.
- *
+ * {@link #read()}.
+ *
*
* The implementation is thread-safe after its properties are set (normal
* singleton behavior).
diff --git a/spring-batch-infrastructure/src/main/java/org/springframework/batch/item/jms/JmsItemWriter.java b/spring-batch-infrastructure/src/main/java/org/springframework/batch/item/jms/JmsItemWriter.java
index 1407d42c3..b23ea3bd3 100644
--- a/spring-batch-infrastructure/src/main/java/org/springframework/batch/item/jms/JmsItemWriter.java
+++ b/spring-batch-infrastructure/src/main/java/org/springframework/batch/item/jms/JmsItemWriter.java
@@ -16,8 +16,6 @@
package org.springframework.batch.item.jms;
-import java.util.List;
-
import org.apache.commons.logging.Log;
import org.apache.commons.logging.LogFactory;
import org.springframework.batch.item.ItemWriter;
@@ -25,11 +23,13 @@ import org.springframework.jms.core.JmsOperations;
import org.springframework.jms.core.JmsTemplate;
import org.springframework.util.Assert;
+import java.util.List;
+
/**
* An {@link ItemWriter} for JMS using a {@link JmsTemplate}. The template
* should have a default destination, which will be used to send items in
- * {@link #write(List)}.
- *
+ * {@link #write(List)}.
+ *
*
* The implementation is thread-safe after its properties are set (normal
* singleton behavior).
diff --git a/spring-batch-infrastructure/src/main/java/org/springframework/batch/item/support/CompositeItemProcessor.java b/spring-batch-infrastructure/src/main/java/org/springframework/batch/item/support/CompositeItemProcessor.java
index c1f9d57d4..c2ce4717b 100644
--- a/spring-batch-infrastructure/src/main/java/org/springframework/batch/item/support/CompositeItemProcessor.java
+++ b/spring-batch-infrastructure/src/main/java/org/springframework/batch/item/support/CompositeItemProcessor.java
@@ -16,17 +16,17 @@
package org.springframework.batch.item.support;
-import java.util.List;
-
import org.springframework.batch.item.ItemProcessor;
import org.springframework.beans.factory.InitializingBean;
import org.springframework.util.Assert;
+import java.util.List;
+
/**
* Composite {@link ItemProcessor} that passes the item through a sequence of
* injected ItemTransformers (return value of previous
- * transformation is the entry value of the next).
- *
+ * transformation is the entry value of the next).
+ *
*
* Note the user is responsible for injecting a chain of {@link ItemProcessor}s
* that conforms to declared input and output types.
diff --git a/spring-batch-infrastructure/src/main/java/org/springframework/batch/item/support/CompositeItemWriter.java b/spring-batch-infrastructure/src/main/java/org/springframework/batch/item/support/CompositeItemWriter.java
index 13db29cc2..d0c874401 100644
--- a/spring-batch-infrastructure/src/main/java/org/springframework/batch/item/support/CompositeItemWriter.java
+++ b/spring-batch-infrastructure/src/main/java/org/springframework/batch/item/support/CompositeItemWriter.java
@@ -16,8 +16,6 @@
package org.springframework.batch.item.support;
-import java.util.List;
-
import org.springframework.batch.item.ExecutionContext;
import org.springframework.batch.item.ItemStream;
import org.springframework.batch.item.ItemStreamException;
@@ -26,9 +24,11 @@ import org.springframework.batch.item.ItemWriter;
import org.springframework.beans.factory.InitializingBean;
import org.springframework.util.Assert;
+import java.util.List;
+
/**
- * Calls a collection of {@link ItemWriter}s in fixed-order sequence.
- *
+ * Calls a collection of {@link ItemWriter}s in fixed-order sequence.
+ *
*
* The implementation is thread-safe if all delegates are thread-safe.
*
diff --git a/spring-batch-infrastructure/src/main/java/org/springframework/batch/item/xml/StaxUtils.java b/spring-batch-infrastructure/src/main/java/org/springframework/batch/item/xml/StaxUtils.java
index 5813d0604..14f9dd4ce 100644
--- a/spring-batch-infrastructure/src/main/java/org/springframework/batch/item/xml/StaxUtils.java
+++ b/spring-batch-infrastructure/src/main/java/org/springframework/batch/item/xml/StaxUtils.java
@@ -16,26 +16,25 @@
package org.springframework.batch.item.xml;
-import java.lang.reflect.Constructor;
-import java.lang.reflect.Method;
-
-import javax.xml.stream.XMLEventReader;
-import javax.xml.stream.XMLEventWriter;
-import javax.xml.transform.Result;
-import javax.xml.transform.Source;
-
import org.apache.commons.logging.Log;
import org.apache.commons.logging.LogFactory;
import org.springframework.util.Assert;
import org.springframework.util.ClassUtils;
+import javax.xml.stream.XMLEventReader;
+import javax.xml.stream.XMLEventWriter;
+import javax.xml.transform.Result;
+import javax.xml.transform.Source;
+import java.lang.reflect.Constructor;
+import java.lang.reflect.Method;
+
/**
* This class provides a little bit of indirection to avoid ugly conditional object creation. It is unfortunately
* a bit redundant assuming a Spring 3.0 environment, but is necessary to work with Spring WS 1.5.x.
- *
+ *
* The returned object determines whether the environment has Spring OXM as included in the Spring 3.x series of relies
* or whether it has Spring OXM from Spring WS 1.5x and factories a StaxSource instance appropriately.
- *
+ *
* As the only class state maintained is to cache java reflection metadata, which is thread-safe, this class is thread-safe.
*
* @author Josh Long
diff --git a/spring-batch-infrastructure/src/main/java/org/springframework/batch/repeat/RepeatListener.java b/spring-batch-infrastructure/src/main/java/org/springframework/batch/repeat/RepeatListener.java
index 3c42f329c..2dc2c114e 100644
--- a/spring-batch-infrastructure/src/main/java/org/springframework/batch/repeat/RepeatListener.java
+++ b/spring-batch-infrastructure/src/main/java/org/springframework/batch/repeat/RepeatListener.java
@@ -59,7 +59,7 @@ public interface RepeatListener {
/**
* Called when a repeat callback fails by throwing an exception. There will
* be one call to this method for each exception thrown during a repeat
- * operation (e.g. a chunk).
+ * operation (e.g. a chunk).
*
* There is no need to re-throw the exception here - that will be done by
* the enclosing framework.
diff --git a/spring-batch-infrastructure/src/main/java/org/springframework/batch/repeat/policy/TimeoutTerminationPolicy.java b/spring-batch-infrastructure/src/main/java/org/springframework/batch/repeat/policy/TimeoutTerminationPolicy.java
index f6e2ce72c..37daf2c70 100644
--- a/spring-batch-infrastructure/src/main/java/org/springframework/batch/repeat/policy/TimeoutTerminationPolicy.java
+++ b/spring-batch-infrastructure/src/main/java/org/springframework/batch/repeat/policy/TimeoutTerminationPolicy.java
@@ -22,7 +22,7 @@ import org.springframework.batch.repeat.context.RepeatContextSupport;
/**
* Termination policy that times out after a fixed period. Allows graceful exit
* from a batch if the latest result comes in after the timeout expires (i.e.
- * does not throw a timeout exception).
+ * does not throw a timeout exception).
*
* N.B. It may often be the case that the batch governed by this policy will be
* transactional, and the transaction might have its own timeout. In this case
diff --git a/spring-batch-infrastructure/src/main/java/org/springframework/batch/repeat/support/RepeatTemplate.java b/spring-batch-infrastructure/src/main/java/org/springframework/batch/repeat/support/RepeatTemplate.java
index cd330e821..2b6efd763 100644
--- a/spring-batch-infrastructure/src/main/java/org/springframework/batch/repeat/support/RepeatTemplate.java
+++ b/spring-batch-infrastructure/src/main/java/org/springframework/batch/repeat/support/RepeatTemplate.java
@@ -16,11 +16,6 @@
package org.springframework.batch.repeat.support;
-import java.util.ArrayList;
-import java.util.Arrays;
-import java.util.Collection;
-import java.util.List;
-
import org.apache.commons.logging.Log;
import org.apache.commons.logging.LogFactory;
import org.springframework.batch.repeat.CompletionPolicy;
@@ -35,18 +30,23 @@ import org.springframework.batch.repeat.exception.ExceptionHandler;
import org.springframework.batch.repeat.policy.DefaultResultCompletionPolicy;
import org.springframework.util.Assert;
+import java.util.ArrayList;
+import java.util.Arrays;
+import java.util.Collection;
+import java.util.List;
+
/**
* Simple implementation and base class for batch templates implementing
* {@link RepeatOperations}. Provides a framework including interceptors and
* policies. Subclasses just need to provide a method that gets the next result
* and one that waits for all the results to be returned from concurrent
- * processes or threads.
+ * processes or threads.
*
* N.B. the template accumulates thrown exceptions during the iteration, and
* they are all processed together when the main loop ends (i.e. finished
* processing the items). Clients that do not want to stop execution when an
* exception is thrown can use a specific {@link CompletionPolicy} that does not
- * finish when exceptions are received. This is not the default behaviour.
+ * finish when exceptions are received. This is not the default behaviour.
*
* Clients that want to take some business action when an exception is thrown by
* the {@link RepeatCallback} can consider using a custom {@link RepeatListener}
diff --git a/spring-batch-infrastructure/src/main/java/org/springframework/batch/repeat/support/ResultHolderResultQueue.java b/spring-batch-infrastructure/src/main/java/org/springframework/batch/repeat/support/ResultHolderResultQueue.java
index 1aecf4f31..7e8c7faad 100644
--- a/spring-batch-infrastructure/src/main/java/org/springframework/batch/repeat/support/ResultHolderResultQueue.java
+++ b/spring-batch-infrastructure/src/main/java/org/springframework/batch/repeat/support/ResultHolderResultQueue.java
@@ -16,14 +16,14 @@
package org.springframework.batch.repeat.support;
+import org.springframework.batch.repeat.RepeatStatus;
+
import java.util.Comparator;
import java.util.NoSuchElementException;
import java.util.concurrent.BlockingQueue;
import java.util.concurrent.PriorityBlockingQueue;
import java.util.concurrent.Semaphore;
-import org.springframework.batch.repeat.RepeatStatus;
-
/**
* An implementation of the {@link ResultQueue} that throttles the number of
* expected results, limiting it to a maximum at any given time.
@@ -100,8 +100,8 @@ public class ResultHolderResultQueue implements ResultQueue
- *
+ * Get the next result as soon as it becomes available.
+ *
* Release result immediately if:
*
*
The timeout for the repoy should be set
* sufficiently long that the remote steps have time to complete.
*
- * @param messagingGateway the {@link MessagingOperations} to set
+ * @param messagingGateway the {@link org.springframework.integration.core.MessagingTemplate} to set
*/
public void setMessagingOperations(MessagingTemplate messagingGateway) {
this.messagingGateway = messagingGateway;
diff --git a/spring-batch-integration/src/test/java/org/springframework/batch/integration/file/ResourceSplitterIntegrationTests.java b/spring-batch-integration/src/test/java/org/springframework/batch/integration/file/ResourceSplitterIntegrationTests.java
index 2a66ad292..e89cdf342 100644
--- a/spring-batch-integration/src/test/java/org/springframework/batch/integration/file/ResourceSplitterIntegrationTests.java
+++ b/spring-batch-integration/src/test/java/org/springframework/batch/integration/file/ResourceSplitterIntegrationTests.java
@@ -53,7 +53,7 @@ public class ResourceSplitterIntegrationTests {
private PollableChannel requests;
/*
- * This is so cool (but see INT-190)...
+ * usually managed elsewhere, e.g. by a {@link TaskExecutor}.
* Implementations may choose to block here, if they need to limit the
* number or rate of tasks being submitted.
*
diff --git a/spring-batch-infrastructure/src/main/java/org/springframework/batch/repeat/support/TaskExecutorRepeatTemplate.java b/spring-batch-infrastructure/src/main/java/org/springframework/batch/repeat/support/TaskExecutorRepeatTemplate.java
index 7e44514cd..c300cfc8c 100644
--- a/spring-batch-infrastructure/src/main/java/org/springframework/batch/repeat/support/TaskExecutorRepeatTemplate.java
+++ b/spring-batch-infrastructure/src/main/java/org/springframework/batch/repeat/support/TaskExecutorRepeatTemplate.java
@@ -27,12 +27,12 @@ import org.springframework.util.Assert;
/**
* Provides {@link RepeatOperations} support including interceptors that can be
- * used to modify or monitor the behaviour at run time.
+ * used to modify or monitor the behaviour at run time.
*
* This implementation is sufficient to be used to configure transactional
* behaviour for each item by making the {@link RepeatCallback} transactional,
* or for the whole batch by making the execute method transactional (but only
- * then if the task executor is synchronous).
+ * then if the task executor is synchronous).
*
* This class is thread-safe if its collaborators are thread-safe (interceptors,
* terminationPolicy, callback). Normally this will be the case, but clients
@@ -41,7 +41,7 @@ import org.springframework.util.Assert;
* is wrapped in the execute method must be thread-safe - often it is based on
* some form of data source, which itself should be both thread-safe and
* transactional (multiple threads could be accessing it at any given time, and
- * each thread would have its own transaction).
+ * each thread would have its own transaction).
*
* @author Dave Syer
*
diff --git a/spring-batch-infrastructure/src/site/apt/changelog.apt b/spring-batch-infrastructure/src/site/apt/changelog.apt
deleted file mode 100644
index 7c41b85a2..000000000
--- a/spring-batch-infrastructure/src/site/apt/changelog.apt
+++ /dev/null
@@ -1,7 +0,0 @@
-Changelog: Spring Batch Infrastructure
-
-* 1.0-M2
-
-** 2007/07/12
-
- * No-one uses this file: we should just switch to auto-generated changelogs?
diff --git a/spring-batch-infrastructure/src/site/apt/index.apt b/spring-batch-infrastructure/src/site/apt/index.apt
deleted file mode 100644
index 1da232bae..000000000
--- a/spring-batch-infrastructure/src/site/apt/index.apt
+++ /dev/null
@@ -1,57 +0,0 @@
- ------
- Spring Batch Infrastructure
- ------
- Dave Syer
- ------
- December 2006
-
-Introduction
-
- This module provides framework code for the Spring Batch project.
- The core interfaces are <<setUp and tearDown methods
@@ -524,7 +524,7 @@ public class FlatFileItemWriterTests {
public void testOpenWithNonWritableFile() throws Exception {
writer = new FlatFileItemWriter
+ * down. Us as a bean definition
*
* Run this class to initialize a database in a running server process.
* Make sure the server is running first by launching the "hsql-server" from the
diff --git a/spring-batch-infrastructure/src/test/resources/org/springframework/batch/item/database/stored-procedure-context.xml b/spring-batch-infrastructure/src/test/resources/org/springframework/batch/item/database/stored-procedure-context.xml
index abb7d264d..3b9eb7210 100644
--- a/spring-batch-infrastructure/src/test/resources/org/springframework/batch/item/database/stored-procedure-context.xml
+++ b/spring-batch-infrastructure/src/test/resources/org/springframework/batch/item/database/stored-procedure-context.xml
@@ -15,7 +15,7 @@
+ * This is so cool (but see INT-190)...
*
* The incoming message is a Resource pattern, and it is converted to the
* correct payload type with Spring's default strategy
diff --git a/spring-batch-integration/template.mf b/spring-batch-integration/template.mf
deleted file mode 100644
index 0782191cd..000000000
--- a/spring-batch-integration/template.mf
+++ /dev/null
@@ -1,25 +0,0 @@
-Bundle-SymbolicName: org.springframework.batch.integration
-Bundle-Name: Spring Batch Integration
-Bundle-Vendor: SpringSource
-Bundle-ManifestVersion: 2
-Import-Template:
- org.springframework.batch.*;version="[2.1.0, 3.0.0)",
- org.springframework.integration.*;version="[2.0.0, 3.0.0)",
- org.springframework.beans.*;version="[3.0.4, 4.0.0)",
- org.springframework.context.*;version="[3.0.4, 4.0.0)",
- org.springframework.core.*;version="[3.0.4, 4.0.0)",
- org.springframework.jdbc.*;version="[3.0.4, 4.0.0)",
- org.springframework.stereotype.*;version="[3.0.4, 4.0.0)",
- org.springframework.scheduling.*;version="[3.0.4, 4.0.0)",
- org.springframework.ui.*;version="[3.0.4, 4.0.0)",
- org.springframework.transaction.*;version="[3.0.4, 4.0.0)",
- org.springframework.web.*;version="[3.0.4, 4.0.0)",
- org.springframework.validation.*;version="[3.0.4, 4.0.0)",
- org.springframework.util;version="[3.0.4, 4.0.0)",
- org.apache.commons.logging;version="[1.1.1, 2.0.0)",
- org.apache.commons.dbcp.*;version="[1.2.2, 2.0.0)";resolution:=optional,
- org.apache.commons.io.*;version="[1.4.0, 2.0.0)";resolution:=optional,
- org.aspectj.*;version="0",
- javax.sql;version="0",
- javax.servlet.*;version="0";resolution:=optional,
- org.w3c.dom.*;version="0"
diff --git a/spring-batch-jsr352-tck/pom.xml b/spring-batch-jsr352-tck/pom.xml
deleted file mode 100644
index 3ef838cd1..000000000
--- a/spring-batch-jsr352-tck/pom.xml
+++ /dev/null
@@ -1,202 +0,0 @@
-
-is*().
+ * is*().
*
* This class is thread-safe (it can be used concurrently by multiple threads)
* as long as the {@link ItemReader} is also thread-safe.
diff --git a/spring-batch-samples/src/main/resources/batch-h2.properties b/spring-batch-samples/src/main/resources/batch-h2.properties
index 925428ced..edfc12008 100644
--- a/spring-batch-samples/src/main/resources/batch-h2.properties
+++ b/spring-batch-samples/src/main/resources/batch-h2.properties
@@ -1,7 +1,7 @@
# Placeholders batch.*
# for H2:
batch.jdbc.driver=org.h2.Driver
-batch.jdbc.url=jdbc:h2:file:target/data/h2
+batch.jdbc.url=jdbc:h2:file:build/data/h2
batch.jdbc.user=sa
batch.jdbc.password=
batch.jdbc.testWhileIdle=false
diff --git a/spring-batch-samples/src/main/resources/jobs/compositeItemWriterSampleJob.xml b/spring-batch-samples/src/main/resources/jobs/compositeItemWriterSampleJob.xml
index 021296173..7a5d1398a 100644
--- a/spring-batch-samples/src/main/resources/jobs/compositeItemWriterSampleJob.xml
+++ b/spring-batch-samples/src/main/resources/jobs/compositeItemWriterSampleJob.xml
@@ -74,7 +74,7 @@