Make RepositoryItemWriter use saveAll by default

Issue spring-projects#3720

Author: fmbenhassine

spring-batch-infrastructure/src/main/java/org/springframework/batch/item/data/RepositoryItemWriter.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2012-2019 the original author or authors.
+ * Copyright 2012-2020 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -36,6 +36,8 @@
  * </p>
  *
  * <p>
+ * By default, this writer will use {@link CrudRepository#saveAll(Iterable)}
+ * to save items, unless another method is selected with {@link #setMethodName(java.lang.String)}.
  * It depends on {@link org.springframework.data.repository.CrudRepository#saveAll(Iterable)}
  * method to store the items for the chunk.  Performance will be determined by that
  * implementation more than this writer.
@@ -107,6 +109,11 @@ protected void doWrite(List<? extends T> items) throws Exception {
 		if (logger.isDebugEnabled()) {
 			logger.debug("Writing to the repository with " + items.size() + " items.");
 		}
+		
+		if (this.methodName == null) {
+			this.repository.saveAll(items);
+			return;
+		}
 
 		MethodInvoker invoker = createMethodInvoker(repository, methodName);
 

spring-batch-infrastructure/src/main/java/org/springframework/batch/item/data/builder/RepositoryItemWriterBuilder.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2017-2018 the original author or authors.
+ * Copyright 2017-2020 the original author or authors.
  *
  *  Licensed under the Apache License, Version 2.0 (the "License");
  *  you may not use this file except in compliance with the License.
@@ -19,6 +19,9 @@
 
 import java.lang.reflect.Method;
 
+import org.apache.commons.logging.Log;
+import org.apache.commons.logging.LogFactory;
+
 import org.springframework.batch.item.data.RepositoryItemWriter;
 import org.springframework.cglib.proxy.Enhancer;
 import org.springframework.cglib.proxy.MethodInterceptor;
@@ -36,6 +39,8 @@
  */
 public class RepositoryItemWriterBuilder<T> {
 
+	private static final Log logger = LogFactory.getLog(RepositoryItemWriterBuilder.class.getName());
+
 	private CrudRepository<T, ?> repository;
 
 	private String methodName;
@@ -104,12 +109,16 @@ public RepositoryItemWriter<T> build() {
 			this.repository = this.repositoryMethodReference.getRepository();
 		}
 
-		Assert.hasText(this.methodName, "methodName is required.");
 		Assert.notNull(this.repository, "repository is required.");
 
 		RepositoryItemWriter<T> writer = new RepositoryItemWriter<>();
-		writer.setMethodName(this.methodName);
 		writer.setRepository(this.repository);
+		if (this.methodName != null) {
+			Assert.hasText(this.methodName, "methodName must not be empty.");
+			writer.setMethodName(this.methodName);
+		} else {
+			logger.debug("No method name provided, CrudRepository.saveAll will be used.");
+		}
 		return writer;
 	}
 

spring-batch-infrastructure/src/test/java/org/springframework/batch/item/data/builder/RepositoryItemWriterBuilderTests.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2017 the original author or authors.
+ * Copyright 2017-2020 the original author or authors.
  *
  *  Licensed under the Apache License, Version 2.0 (the "License");
  *  you may not use this file except in compliance with the License.
@@ -34,6 +34,7 @@
 
 /**
  * @author Glenn Renfro
+ * @author Mahmoud Ben Hassine
  */
 public class RepositoryItemWriterBuilderTests {
 	@Mock
@@ -58,15 +59,18 @@ public void testNullRepository() throws Exception {
 	}
 
 	@Test
-	public void testEmptyMethodName() throws Exception {
+	public void testEmptyMethodName() {
 		try {
-			new RepositoryItemWriterBuilder<String>().repository(this.repository).build();
+			new RepositoryItemWriterBuilder<String>()
+					.repository(this.repository)
+					.methodName("")
+					.build();
 
 			fail("IllegalArgumentException should have been thrown");
 		}
 		catch (IllegalArgumentException iae) {
 			assertEquals("IllegalArgumentException message did not match the expected result.",
-					"methodName is required.", iae.getMessage());
+					"methodName must not be empty.", iae.getMessage());
 		}
 	}