RESOLVED - BATCH-930: rename ItemSkipPolicy
ItemSkipPolicy => SkipPolicy
This commit is contained in:
@@ -9,7 +9,7 @@ import java.util.Map.Entry;
|
||||
|
||||
import org.springframework.batch.core.SkipListener;
|
||||
import org.springframework.batch.core.StepContribution;
|
||||
import org.springframework.batch.core.step.skip.ItemSkipPolicy;
|
||||
import org.springframework.batch.core.step.skip.SkipPolicy;
|
||||
import org.springframework.batch.core.step.skip.SkipListenerFailedException;
|
||||
import org.springframework.batch.item.ItemProcessor;
|
||||
import org.springframework.batch.item.ItemReader;
|
||||
@@ -48,18 +48,18 @@ public abstract class AbstractFaultTolerantChunkOrientedTasklet<I, O> extends Ab
|
||||
|
||||
final private RepeatOperations repeatOperations;
|
||||
|
||||
final private ItemSkipPolicy writeSkipPolicy;
|
||||
final private SkipPolicy writeSkipPolicy;
|
||||
|
||||
final private ItemSkipPolicy processSkipPolicy;
|
||||
final private SkipPolicy processSkipPolicy;
|
||||
|
||||
final private ItemSkipPolicy readSkipPolicy;
|
||||
final private SkipPolicy readSkipPolicy;
|
||||
|
||||
final private Classifier<Throwable, Boolean> rollbackClassifier;
|
||||
|
||||
public AbstractFaultTolerantChunkOrientedTasklet(ItemReader<? extends I> itemReader,
|
||||
ItemProcessor<? super I, ? extends O> itemProcessor, ItemWriter<? super O> itemWriter,
|
||||
RetryOperations retryOperations, ItemSkipPolicy readSkipPolicy, ItemSkipPolicy processSkipPolicy,
|
||||
ItemSkipPolicy writeSkipPolicy, Classifier<Throwable, Boolean> rollbackClassifier,
|
||||
RetryOperations retryOperations, SkipPolicy readSkipPolicy, SkipPolicy processSkipPolicy,
|
||||
SkipPolicy writeSkipPolicy, Classifier<Throwable, Boolean> rollbackClassifier,
|
||||
RepeatOperations repeatTemplate) {
|
||||
|
||||
super(itemReader, itemProcessor, itemWriter);
|
||||
@@ -71,7 +71,7 @@ public abstract class AbstractFaultTolerantChunkOrientedTasklet<I, O> extends Ab
|
||||
this.repeatOperations = repeatTemplate;
|
||||
}
|
||||
|
||||
protected ItemSkipPolicy getReadSkipPolicy() {
|
||||
protected SkipPolicy getReadSkipPolicy() {
|
||||
return readSkipPolicy;
|
||||
}
|
||||
|
||||
|
||||
@@ -21,7 +21,7 @@ import java.util.Map;
|
||||
|
||||
import org.springframework.batch.core.ExitStatus;
|
||||
import org.springframework.batch.core.StepContribution;
|
||||
import org.springframework.batch.core.step.skip.ItemSkipPolicy;
|
||||
import org.springframework.batch.core.step.skip.SkipPolicy;
|
||||
import org.springframework.batch.core.step.skip.NonSkippableReadException;
|
||||
import org.springframework.batch.item.ItemProcessor;
|
||||
import org.springframework.batch.item.ItemReader;
|
||||
@@ -59,8 +59,8 @@ public class FaultTolerantChunkOrientedTasklet<I, O> extends AbstractFaultTolera
|
||||
public FaultTolerantChunkOrientedTasklet(ItemReader<? extends I> itemReader,
|
||||
ItemProcessor<? super I, ? extends O> itemProcessor, ItemWriter<? super O> itemWriter,
|
||||
RepeatOperations chunkOperations, RetryOperations retryTemplate,
|
||||
Classifier<Throwable, Boolean> rollbackClassifier, ItemSkipPolicy readSkipPolicy,
|
||||
ItemSkipPolicy writeSkipPolicy, ItemSkipPolicy processSkipPolicy) {
|
||||
Classifier<Throwable, Boolean> rollbackClassifier, SkipPolicy readSkipPolicy,
|
||||
SkipPolicy writeSkipPolicy, SkipPolicy processSkipPolicy) {
|
||||
|
||||
super(itemReader, itemProcessor, itemWriter, retryTemplate, readSkipPolicy, processSkipPolicy, writeSkipPolicy,
|
||||
rollbackClassifier, chunkOperations);
|
||||
|
||||
@@ -6,7 +6,7 @@ import java.util.HashMap;
|
||||
import java.util.HashSet;
|
||||
import java.util.List;
|
||||
|
||||
import org.springframework.batch.core.step.skip.ItemSkipPolicy;
|
||||
import org.springframework.batch.core.step.skip.SkipPolicy;
|
||||
import org.springframework.batch.core.step.skip.LimitCheckingItemSkipPolicy;
|
||||
import org.springframework.batch.core.step.skip.NonSkippableReadException;
|
||||
import org.springframework.batch.core.step.skip.SkipLimitExceededException;
|
||||
@@ -257,10 +257,10 @@ public class FaultTolerantStepFactoryBean<T, S> extends SimpleStepFactoryBean<T,
|
||||
|
||||
List<Class<? extends Throwable>> exceptions = new ArrayList<Class<? extends Throwable>>(
|
||||
skippableExceptionClasses);
|
||||
ItemSkipPolicy readSkipPolicy = new LimitCheckingItemSkipPolicy(skipLimit, skippableExceptionClasses,
|
||||
SkipPolicy readSkipPolicy = new LimitCheckingItemSkipPolicy(skipLimit, skippableExceptionClasses,
|
||||
new ArrayList<Class<? extends Throwable>>(fatalExceptionClasses));
|
||||
exceptions.addAll(new ArrayList<Class<? extends Throwable>>(retryableExceptionClasses));
|
||||
ItemSkipPolicy writeSkipPolicy = new LimitCheckingItemSkipPolicy(skipLimit, exceptions,
|
||||
SkipPolicy writeSkipPolicy = new LimitCheckingItemSkipPolicy(skipLimit, exceptions,
|
||||
new ArrayList<Class<? extends Throwable>>(fatalExceptionClasses));
|
||||
|
||||
if (isReaderTransactionalQueue) {
|
||||
|
||||
@@ -8,7 +8,7 @@ import java.util.Set;
|
||||
|
||||
import org.springframework.batch.core.ExitStatus;
|
||||
import org.springframework.batch.core.StepContribution;
|
||||
import org.springframework.batch.core.step.skip.ItemSkipPolicy;
|
||||
import org.springframework.batch.core.step.skip.SkipPolicy;
|
||||
import org.springframework.batch.core.step.skip.SkipListenerFailedException;
|
||||
import org.springframework.batch.item.ItemProcessor;
|
||||
import org.springframework.batch.item.ItemReader;
|
||||
@@ -41,8 +41,8 @@ public class NonbufferingFaultTolerantChunkOrientedTasklet<I, O> extends
|
||||
public NonbufferingFaultTolerantChunkOrientedTasklet(ItemReader<? extends I> itemReader,
|
||||
ItemProcessor<? super I, ? extends O> itemProcessor, ItemWriter<? super O> itemWriter,
|
||||
RepeatOperations chunkOperations, RetryOperations retryTemplate,
|
||||
Classifier<Throwable, Boolean> rollbackClassifier, ItemSkipPolicy readSkipPolicy,
|
||||
ItemSkipPolicy writeSkipPolicy, ItemSkipPolicy processSkipPolicy) {
|
||||
Classifier<Throwable, Boolean> rollbackClassifier, SkipPolicy readSkipPolicy,
|
||||
SkipPolicy writeSkipPolicy, SkipPolicy processSkipPolicy) {
|
||||
|
||||
super(itemReader, itemProcessor, itemWriter, retryTemplate, readSkipPolicy, processSkipPolicy, writeSkipPolicy,
|
||||
rollbackClassifier, chunkOperations);
|
||||
|
||||
@@ -17,13 +17,13 @@ package org.springframework.batch.core.step.skip;
|
||||
|
||||
|
||||
/**
|
||||
* Implementation of the {@link ItemSkipPolicy} interface that
|
||||
* Implementation of the {@link SkipPolicy} interface that
|
||||
* will always return that an item should be skipped.
|
||||
*
|
||||
* @author Ben Hale
|
||||
* @author Lucas Ward
|
||||
*/
|
||||
public class AlwaysSkipItemSkipPolicy implements ItemSkipPolicy {
|
||||
public class AlwaysSkipItemSkipPolicy implements SkipPolicy {
|
||||
|
||||
public boolean shouldSkip(Throwable t, int skipCount) {
|
||||
return true;
|
||||
|
||||
@@ -26,7 +26,7 @@ import org.springframework.batch.support.Classifier;
|
||||
|
||||
/**
|
||||
* <p>
|
||||
* {@link ItemSkipPolicy} that determines whether or not reading should continue
|
||||
* {@link SkipPolicy} that determines whether or not reading should continue
|
||||
* based upon how many items have been skipped. This is extremely useful
|
||||
* behavior, as it allows you to skip records, but will throw a
|
||||
* {@link SkipLimitExceededException} if a set limit has been exceeded. For
|
||||
@@ -52,7 +52,7 @@ import org.springframework.batch.support.Classifier;
|
||||
* @author Lucas Ward
|
||||
* @author Robert Kasanicky
|
||||
*/
|
||||
public class LimitCheckingItemSkipPolicy implements ItemSkipPolicy {
|
||||
public class LimitCheckingItemSkipPolicy implements SkipPolicy {
|
||||
|
||||
private final int skipLimit;
|
||||
|
||||
|
||||
@@ -17,12 +17,12 @@ package org.springframework.batch.core.step.skip;
|
||||
|
||||
|
||||
/**
|
||||
* {@link ItemSkipPolicy} implementation that always returns false,
|
||||
* {@link SkipPolicy} implementation that always returns false,
|
||||
* indicating that an item should not be skipped.
|
||||
*
|
||||
* @author Lucas Ward
|
||||
*/
|
||||
public class NeverSkipItemSkipPolicy implements ItemSkipPolicy{
|
||||
public class NeverSkipItemSkipPolicy implements SkipPolicy{
|
||||
|
||||
public boolean shouldSkip(Throwable t, int skipCount) {
|
||||
return false;
|
||||
|
||||
@@ -21,7 +21,7 @@ package org.springframework.batch.core.step.skip;
|
||||
* @author Lucas Ward
|
||||
* @author Dave Syer
|
||||
*/
|
||||
public interface ItemSkipPolicy {
|
||||
public interface SkipPolicy {
|
||||
|
||||
/**
|
||||
* Returns true or false, indicating whether or not processing should
|
||||
@@ -33,7 +33,7 @@ import org.springframework.batch.core.SkipListener;
|
||||
import org.springframework.batch.core.StepContribution;
|
||||
import org.springframework.batch.core.StepExecution;
|
||||
import org.springframework.batch.core.scope.ChunkContext;
|
||||
import org.springframework.batch.core.step.skip.ItemSkipPolicy;
|
||||
import org.springframework.batch.core.step.skip.SkipPolicy;
|
||||
import org.springframework.batch.core.step.skip.SkipLimitExceededException;
|
||||
import org.springframework.batch.item.ItemProcessor;
|
||||
import org.springframework.batch.item.ItemReader;
|
||||
@@ -95,7 +95,7 @@ public class FaultTolerantChunkOrientedTaskletTests {
|
||||
}
|
||||
};
|
||||
|
||||
private ItemSkipPolicy readSkipPolicy = new ItemSkipPolicy() {
|
||||
private SkipPolicy readSkipPolicy = new SkipPolicy() {
|
||||
public boolean shouldSkip(Throwable t, int skipCount) throws SkipLimitExceededException {
|
||||
if (skipCount < skipLimit) {
|
||||
return true;
|
||||
@@ -104,7 +104,7 @@ public class FaultTolerantChunkOrientedTaskletTests {
|
||||
}
|
||||
};
|
||||
|
||||
private ItemSkipPolicy writeSkipPolicy = readSkipPolicy;
|
||||
private SkipPolicy writeSkipPolicy = readSkipPolicy;
|
||||
|
||||
@Before
|
||||
public void setUp() {
|
||||
|
||||
Reference in New Issue
Block a user