IN PROGRESS - issue BATCH-212: Merge InputSource with ItemProvider
http://opensource.atlassian.com/projects/spring/browse/BATCH-212 Rename class and method names (Provider -> Reader)
This commit is contained in:
@@ -168,7 +168,7 @@ public class ItemOrientedTasklet implements Tasklet, Skippable,
|
||||
/**
|
||||
* @param itemProvider
|
||||
*/
|
||||
public void setItemProvider(ItemReader itemProvider) {
|
||||
public void setItemReader(ItemReader itemProvider) {
|
||||
this.itemProvider = itemProvider;
|
||||
}
|
||||
|
||||
|
||||
@@ -104,7 +104,7 @@ public class SimpleJobTests extends TestCase {
|
||||
return true;
|
||||
}
|
||||
});
|
||||
module.setItemProvider(provider);
|
||||
module.setItemReader(provider);
|
||||
module.setItemProcessor(processor);
|
||||
module.afterPropertiesSet();
|
||||
return module;
|
||||
|
||||
@@ -66,7 +66,7 @@ public class DefaultStepExecutorTests extends TestCase {
|
||||
|
||||
private RepeatTemplate template;
|
||||
|
||||
private ItemReader getProvider(String[] args) {
|
||||
private ItemReader getReader(String[] args) {
|
||||
return new ListItemReader(Arrays.asList(args));
|
||||
}
|
||||
|
||||
@@ -78,7 +78,7 @@ public class DefaultStepExecutorTests extends TestCase {
|
||||
private Tasklet getTasklet(String[] strings) throws Exception {
|
||||
ItemOrientedTasklet module = new ItemOrientedTasklet();
|
||||
module.setItemProcessor(processor);
|
||||
module.setItemProvider(getProvider(strings));
|
||||
module.setItemReader(getReader(strings));
|
||||
module.afterPropertiesSet();
|
||||
return module;
|
||||
}
|
||||
|
||||
@@ -73,7 +73,7 @@ public class ItemOrientedTaskletTests extends TestCase {
|
||||
module = new ItemOrientedTasklet();
|
||||
|
||||
// set up module
|
||||
module.setItemProvider(itemProvider);
|
||||
module.setItemReader(itemProvider);
|
||||
module.setItemProcessor(itemProcessor);
|
||||
|
||||
module.afterPropertiesSet();
|
||||
@@ -145,14 +145,14 @@ public class ItemOrientedTaskletTests extends TestCase {
|
||||
}
|
||||
|
||||
public void testSkippableReader() throws Exception {
|
||||
module.setItemProvider(new SkippableItemReader());
|
||||
module.setItemReader(new SkippableItemReader());
|
||||
module.setItemRecoverer(null);
|
||||
module.skip();
|
||||
assertEquals(1, list.size());
|
||||
}
|
||||
|
||||
public void testSkippablReaderProcessor() throws Exception {
|
||||
module.setItemProvider(new SkippableItemReader());
|
||||
module.setItemReader(new SkippableItemReader());
|
||||
module.setItemProcessor(new SkippableItemProcessor());
|
||||
module.setItemRecoverer(null);
|
||||
module.skip();
|
||||
@@ -160,7 +160,7 @@ public class ItemOrientedTaskletTests extends TestCase {
|
||||
}
|
||||
|
||||
public void testStatisticsProvider() throws Exception {
|
||||
module.setItemProvider(new SkippableItemReader());
|
||||
module.setItemReader(new SkippableItemReader());
|
||||
Properties stats = module.getStatistics();
|
||||
assertEquals(1, stats.size());
|
||||
assertEquals("bar", stats.getProperty("foo"));
|
||||
@@ -174,7 +174,7 @@ public class ItemOrientedTaskletTests extends TestCase {
|
||||
}
|
||||
|
||||
public void testStatisticsProviderProcessor() throws Exception {
|
||||
module.setItemProvider(new SkippableItemReader());
|
||||
module.setItemReader(new SkippableItemReader());
|
||||
module.setItemProcessor(new SkippableItemProcessor());
|
||||
Properties stats = module.getStatistics();
|
||||
assertEquals(2, stats.size());
|
||||
@@ -184,7 +184,7 @@ public class ItemOrientedTaskletTests extends TestCase {
|
||||
|
||||
public void testStatisticsProviderProcessorMergeDuplicates()
|
||||
throws Exception {
|
||||
module.setItemProvider(new SkippableItemReader());
|
||||
module.setItemReader(new SkippableItemReader());
|
||||
module.setItemProcessor(new SkippableItemProcessor(
|
||||
"foo=bar\nspam=bucket"));
|
||||
Properties stats = module.getStatistics();
|
||||
@@ -207,7 +207,7 @@ public class ItemOrientedTaskletTests extends TestCase {
|
||||
}
|
||||
});
|
||||
|
||||
module.setItemProvider(new AbstractItemReader() {
|
||||
module.setItemReader(new AbstractItemReader() {
|
||||
public Object read() throws Exception {
|
||||
return "bar";
|
||||
}
|
||||
@@ -245,7 +245,7 @@ public class ItemOrientedTaskletTests extends TestCase {
|
||||
return true;
|
||||
}
|
||||
});
|
||||
module.setItemProvider(new AbstractItemReader() {
|
||||
module.setItemReader(new AbstractItemReader() {
|
||||
public Object read() throws Exception {
|
||||
return "foo";
|
||||
}
|
||||
@@ -278,7 +278,7 @@ public class ItemOrientedTaskletTests extends TestCase {
|
||||
}
|
||||
|
||||
public void testInitialisationWithNullProvider() throws Exception {
|
||||
module.setItemProvider(null);
|
||||
module.setItemReader(null);
|
||||
try {
|
||||
module.afterPropertiesSet();
|
||||
} catch (IllegalArgumentException e) {
|
||||
|
||||
@@ -102,7 +102,7 @@ public class RestartableItemOrientedTaskletTests extends TestCase {
|
||||
|
||||
// create and set up module
|
||||
module = new RestartableItemOrientedTasklet();
|
||||
module.setItemProvider(itemProvider);
|
||||
module.setItemReader(itemProvider);
|
||||
module.setItemProcessor(itemProcessor);
|
||||
|
||||
// get restart data
|
||||
@@ -120,7 +120,7 @@ public class RestartableItemOrientedTaskletTests extends TestCase {
|
||||
|
||||
// create and set up module
|
||||
module = new RestartableItemOrientedTasklet();
|
||||
module.setItemProvider(itemProvider);
|
||||
module.setItemReader(itemProvider);
|
||||
module.setItemProcessor(itemProcessor);
|
||||
|
||||
// get restart data
|
||||
@@ -135,7 +135,7 @@ public class RestartableItemOrientedTaskletTests extends TestCase {
|
||||
|
||||
// create and set up module
|
||||
module = new RestartableItemOrientedTasklet();
|
||||
module.setItemProvider(null);
|
||||
module.setItemReader(null);
|
||||
module.setItemProcessor(null);
|
||||
|
||||
// get restart data
|
||||
|
||||
@@ -17,7 +17,7 @@
|
||||
<constructor-arg>
|
||||
<bean
|
||||
class="org.springframework.batch.execution.tasklet.ItemOrientedTasklet">
|
||||
<property name="itemProvider">
|
||||
<property name="itemReader">
|
||||
<bean
|
||||
class="org.springframework.batch.item.provider.ListItemReader">
|
||||
<constructor-arg value="foo,bar,spam"/>
|
||||
|
||||
@@ -19,7 +19,7 @@
|
||||
<constructor-arg>
|
||||
<bean
|
||||
class="org.springframework.batch.execution.tasklet.ItemOrientedTasklet">
|
||||
<property name="itemProvider">
|
||||
<property name="itemReader">
|
||||
<bean
|
||||
class="org.springframework.batch.item.provider.ListItemReader">
|
||||
<constructor-arg value="foo,bar,spam"/>
|
||||
|
||||
@@ -127,7 +127,7 @@ public class ItemReaderRetryCallback implements RetryCallback {
|
||||
*
|
||||
* @return the {@link ItemReader} instance.
|
||||
*/
|
||||
public ItemReader getProvider() {
|
||||
public ItemReader getReader() {
|
||||
return provider;
|
||||
}
|
||||
|
||||
|
||||
@@ -152,7 +152,7 @@ public class ItemReaderRetryPolicy extends AbstractStatefulRetryPolicy {
|
||||
public ItemProviderRetryContext(ItemReaderRetryCallback callback) {
|
||||
super(RetrySynchronizationManager.getContext());
|
||||
item = callback.next(this);
|
||||
this.provider = callback.getProvider();
|
||||
this.provider = callback.getReader();
|
||||
this.recoverer = callback.getRecoverer();
|
||||
}
|
||||
|
||||
|
||||
@@ -157,7 +157,7 @@ public class ItemReaderRetryCallbackTests extends TestCase {
|
||||
}
|
||||
|
||||
public void testGetKey() throws Exception {
|
||||
assertEquals("key0", callback.getProvider().getKey("foo"));
|
||||
assertEquals("key0", callback.getReader().getKey("foo"));
|
||||
}
|
||||
|
||||
public void testRecoverWithoutSession() throws Exception {
|
||||
|
||||
@@ -17,7 +17,7 @@
|
||||
<property name="tasklet">
|
||||
<bean
|
||||
class="org.springframework.batch.execution.tasklet.RestartableItemOrientedTasklet">
|
||||
<property name="itemProvider">
|
||||
<property name="itemReader">
|
||||
<bean
|
||||
class="org.springframework.batch.item.provider.ValidatingItemReader">
|
||||
<property name="itemReader" ref="fileInputTemplate" />
|
||||
@@ -36,7 +36,7 @@
|
||||
<property name="tasklet">
|
||||
<bean
|
||||
class="org.springframework.batch.execution.tasklet.RestartableItemOrientedTasklet">
|
||||
<property name="itemProvider">
|
||||
<property name="itemReader">
|
||||
<bean
|
||||
class="org.springframework.batch.item.provider.DelegatingItemReader">
|
||||
<property name="itemReader" ref="fileInputTemplate2" />
|
||||
|
||||
@@ -16,7 +16,7 @@
|
||||
<property name="tasklet">
|
||||
<bean
|
||||
class="org.springframework.batch.execution.tasklet.RestartableItemOrientedTasklet">
|
||||
<property name="itemProvider">
|
||||
<property name="itemReader">
|
||||
<bean
|
||||
class="org.springframework.batch.item.provider.ValidatingItemReader">
|
||||
<property name="itemReader" ref="fileInputTemplate" />
|
||||
|
||||
@@ -23,7 +23,7 @@
|
||||
<property name="tasklet">
|
||||
<bean
|
||||
class="org.springframework.batch.execution.tasklet.ItemOrientedTasklet">
|
||||
<property name="itemProvider">
|
||||
<property name="itemReader">
|
||||
<bean class="org.springframework.batch.item.provider.ItemReaderAdapter">
|
||||
<property name="targetObject" ref="delegatingObject" />
|
||||
<property name="targetMethod" value="getData" />
|
||||
|
||||
@@ -16,7 +16,7 @@
|
||||
<property name="tasklet">
|
||||
<bean
|
||||
class="org.springframework.batch.execution.tasklet.RestartableItemOrientedTasklet">
|
||||
<property name="itemProvider">
|
||||
<property name="itemReader">
|
||||
<bean
|
||||
class="org.springframework.batch.item.provider.ValidatingItemReader">
|
||||
<property name="itemReader" ref="fileInputTemplate" />
|
||||
|
||||
@@ -27,7 +27,7 @@
|
||||
<property name="tasklet">
|
||||
<bean
|
||||
class="org.springframework.batch.execution.tasklet.RestartableItemOrientedTasklet">
|
||||
<property name="itemProvider">
|
||||
<property name="itemReader">
|
||||
<bean
|
||||
class="org.springframework.batch.item.provider.DelegatingItemReader">
|
||||
<property name="itemReader"
|
||||
@@ -57,7 +57,7 @@
|
||||
<property name="tasklet">
|
||||
<bean
|
||||
class="org.springframework.batch.execution.tasklet.RestartableItemOrientedTasklet">
|
||||
<property name="itemProvider">
|
||||
<property name="itemReader">
|
||||
<bean
|
||||
class="org.springframework.batch.item.provider.DelegatingItemReader">
|
||||
<property name="itemReader"
|
||||
@@ -87,7 +87,7 @@
|
||||
<property name="tasklet">
|
||||
<bean
|
||||
class="org.springframework.batch.execution.tasklet.RestartableItemOrientedTasklet">
|
||||
<property name="itemProvider">
|
||||
<property name="itemReader">
|
||||
<bean
|
||||
class="org.springframework.batch.item.provider.DelegatingItemReader">
|
||||
<property name="itemReader"
|
||||
|
||||
@@ -21,7 +21,7 @@
|
||||
<property name="tasklet">
|
||||
<bean
|
||||
class="org.springframework.batch.execution.tasklet.RestartableItemOrientedTasklet">
|
||||
<property name="itemProvider">
|
||||
<property name="itemReader">
|
||||
<bean
|
||||
class="org.springframework.batch.item.provider.DelegatingItemReader">
|
||||
<property name="itemReader"
|
||||
|
||||
@@ -20,7 +20,7 @@
|
||||
<property name="tasklet">
|
||||
<bean
|
||||
class="org.springframework.batch.execution.tasklet.RestartableItemOrientedTasklet">
|
||||
<property name="itemProvider">
|
||||
<property name="itemReader">
|
||||
<bean class="org.springframework.batch.item.provider.DelegatingItemReader">
|
||||
<property name="itemReader" ref="ibatisItemReader" />
|
||||
</bean>
|
||||
|
||||
@@ -16,7 +16,7 @@
|
||||
<property name="tasklet">
|
||||
<bean
|
||||
class="org.springframework.batch.execution.tasklet.RestartableItemOrientedTasklet">
|
||||
<property name="itemProvider">
|
||||
<property name="itemReader">
|
||||
<bean
|
||||
class="org.springframework.batch.item.provider.AggregateItemReader">
|
||||
<property name="itemReader"
|
||||
|
||||
@@ -19,7 +19,7 @@
|
||||
<property name="tasklet">
|
||||
<bean
|
||||
class="org.springframework.batch.execution.tasklet.RestartableItemOrientedTasklet">
|
||||
<property name="itemProvider">
|
||||
<property name="itemReader">
|
||||
<bean class="org.springframework.batch.sample.item.provider.OrderItemReader">
|
||||
<property name="itemReader" ref="fileInputTemplate" />
|
||||
<property name="headerMapper" ref="headerFieldSetMapper" />
|
||||
|
||||
@@ -22,7 +22,7 @@
|
||||
<property name="tasklet">
|
||||
<bean
|
||||
class="org.springframework.batch.execution.tasklet.RestartableItemOrientedTasklet">
|
||||
<property name="itemProvider">
|
||||
<property name="itemReader">
|
||||
<bean
|
||||
class="org.springframework.batch.item.provider.DelegatingItemReader">
|
||||
<property name="itemReader"
|
||||
@@ -57,7 +57,7 @@
|
||||
<property name="tasklet">
|
||||
<bean
|
||||
class="org.springframework.batch.execution.tasklet.ItemOrientedTasklet">
|
||||
<property name="itemProvider">
|
||||
<property name="itemReader">
|
||||
<bean
|
||||
class="org.springframework.batch.sample.item.provider.StagingItemReader"
|
||||
scope="step">
|
||||
@@ -110,7 +110,7 @@
|
||||
<property name="tasklet">
|
||||
<bean
|
||||
class="org.springframework.batch.execution.tasklet.RestartableItemOrientedTasklet">
|
||||
<property name="itemProvider">
|
||||
<property name="itemReader">
|
||||
<bean
|
||||
class="org.springframework.batch.item.provider.DelegatingItemReader">
|
||||
<property name="itemReader"
|
||||
@@ -140,7 +140,7 @@
|
||||
<property name="tasklet">
|
||||
<bean
|
||||
class="org.springframework.batch.execution.tasklet.RestartableItemOrientedTasklet">
|
||||
<property name="itemProvider">
|
||||
<property name="itemReader">
|
||||
<bean
|
||||
class="org.springframework.batch.item.provider.DelegatingItemReader">
|
||||
<property name="itemReader"
|
||||
|
||||
@@ -16,7 +16,7 @@
|
||||
<bean id="step1" parent="simpleStep">
|
||||
<property name="tasklet">
|
||||
<bean class="org.springframework.batch.sample.tasklet.ExceptionRestartableTasklet">
|
||||
<property name="itemProvider">
|
||||
<property name="itemReader">
|
||||
<bean
|
||||
class="org.springframework.batch.item.provider.ValidatingItemReader">
|
||||
<property name="itemReader" ref="fileItemReader" />
|
||||
|
||||
@@ -44,7 +44,7 @@
|
||||
<property name="tasklet">
|
||||
<bean
|
||||
class="org.springframework.batch.execution.tasklet.RestartableItemOrientedTasklet">
|
||||
<property name="itemProvider">
|
||||
<property name="itemReader">
|
||||
<bean
|
||||
class="org.springframework.batch.item.provider.ValidatingItemReader">
|
||||
<property name="itemReader"
|
||||
@@ -65,7 +65,7 @@
|
||||
<property name="tasklet">
|
||||
<bean
|
||||
class="org.springframework.batch.execution.tasklet.RestartableItemOrientedTasklet">
|
||||
<property name="itemProvider">
|
||||
<property name="itemReader">
|
||||
<bean
|
||||
class="org.springframework.batch.item.provider.DelegatingItemReader"
|
||||
p:item-reader-ref="tradeSqlItemReader" />
|
||||
|
||||
@@ -20,7 +20,7 @@
|
||||
<property name="tasklet">
|
||||
<bean
|
||||
class="org.springframework.batch.execution.tasklet.RestartableItemOrientedTasklet">
|
||||
<property name="itemProvider">
|
||||
<property name="itemReader">
|
||||
<bean
|
||||
class="org.springframework.batch.item.provider.ValidatingItemReader">
|
||||
<property name="itemReader"
|
||||
@@ -41,7 +41,7 @@
|
||||
<property name="tasklet">
|
||||
<bean
|
||||
class="org.springframework.batch.execution.tasklet.RestartableItemOrientedTasklet">
|
||||
<property name="itemProvider">
|
||||
<property name="itemReader">
|
||||
<bean
|
||||
class="org.springframework.batch.item.provider.DelegatingItemReader">
|
||||
<property name="itemReader"
|
||||
@@ -60,7 +60,7 @@
|
||||
<property name="tasklet">
|
||||
<bean
|
||||
class="org.springframework.batch.execution.tasklet.RestartableItemOrientedTasklet">
|
||||
<property name="itemProvider">
|
||||
<property name="itemReader">
|
||||
<bean
|
||||
class="org.springframework.batch.item.provider.DelegatingItemReader">
|
||||
<property name="itemReader" ref="customerSqlItemReader" />
|
||||
|
||||
@@ -18,7 +18,7 @@
|
||||
<property name="tasklet">
|
||||
<bean
|
||||
class="org.springframework.batch.execution.tasklet.RestartableItemOrientedTasklet">
|
||||
<property name="itemProvider">
|
||||
<property name="itemReader">
|
||||
<bean class="org.springframework.batch.item.provider.DelegatingItemReader">
|
||||
<property name="itemReader">
|
||||
<bean class="org.springframework.batch.io.file.support.StaxEventReaderItemReader" scope="step">
|
||||
|
||||
@@ -56,7 +56,7 @@ public class StagingItemReaderTests extends
|
||||
getJdbcTemplate().update("DELETE FROM BATCH_STAGING");
|
||||
}
|
||||
|
||||
public void testProviderUpdatesProcessIndicator() throws Exception {
|
||||
public void testReaderUpdatesProcessIndicator() throws Exception {
|
||||
|
||||
long id = getJdbcTemplate().queryForLong(
|
||||
"SELECT MIN(ID) from BATCH_STAGING where JOB_ID=?",
|
||||
@@ -77,7 +77,7 @@ public class StagingItemReaderTests extends
|
||||
}
|
||||
|
||||
public void testUpdateProcessIndicatorAfterCommit() throws Exception {
|
||||
testProviderUpdatesProcessIndicator();
|
||||
testReaderUpdatesProcessIndicator();
|
||||
setComplete();
|
||||
endTransaction();
|
||||
startNewTransaction();
|
||||
|
||||
@@ -38,7 +38,7 @@ public class ExceptionRestartableTaskletTests extends TestCase {
|
||||
module.setItemProcessor(itemProcessor);
|
||||
module.setThrowExceptionOnRecordNumber(ITER_COUNT + 1);
|
||||
|
||||
module.setItemProvider(new ListItemReader(new ArrayList() {{
|
||||
module.setItemReader(new ListItemReader(new ArrayList() {{
|
||||
add("a");
|
||||
add("b");
|
||||
add("c");
|
||||
|
||||
Reference in New Issue
Block a user