Polishing.

Formatting.
Fixing SQL scripts.

Original pull request #1956
See #831

Author: schauder

spring-data-jdbc/src/main/java/org/springframework/data/jdbc/core/AggregateChangeExecutor.java

@@ -78,6 +78,7 @@ <T> void executeDelete(AggregateChange<T> aggregateChange) {
 	}
 
 	private void execute(DbAction<?> action, JdbcAggregateChangeExecutionContext executionContext) {
+
 		if (action instanceof DbAction.InsertRoot<?> insertRoot) {
 			executionContext.executeInsertRoot(insertRoot);
 		} else if (action instanceof DbAction.BatchInsertRoot<?> batchInsertRoot) {

spring-data-jdbc/src/test/java/org/springframework/data/jdbc/core/AbstractJdbcAggregateTemplateIntegrationTests.java

@@ -53,7 +53,6 @@
 import org.springframework.data.jdbc.testing.TestConfiguration;
 import org.springframework.data.jdbc.testing.TestDatabaseFeatures;
 import org.springframework.data.mapping.context.InvalidPersistentPropertyPath;
-import org.springframework.data.relational.core.conversion.DbActionExecutionException;
 import org.springframework.data.relational.core.mapping.Column;
 import org.springframework.data.relational.core.mapping.Embedded;
 import org.springframework.data.relational.core.mapping.InsertOnlyProperty;
@@ -317,7 +316,7 @@ void saveAndLoadManeEntitiesWithReferenceEntityLikeStream() {
 
 		Stream<LegoSet> streamable = template.streamAll(LegoSet.class);
 
-		assertThat(streamable)
+		assertThat(streamable) //
 				.extracting("id", "manual.id", "manual.content") //
 				.containsExactly(tuple(legoSet.id, legoSet.manual.id, legoSet.manual.content));
 	}
@@ -730,7 +729,7 @@ void saveAndLoadAnEntityWithArray() {
 	void saveAndLoadAnEntityWithEmptyArray() {
 
 		ArrayOwner arrayOwner = new ArrayOwner();
-		arrayOwner.digits = new String[] { };
+		arrayOwner.digits = new String[] {};
 
 		ArrayOwner saved = template.save(arrayOwner);
 
@@ -854,7 +853,7 @@ void saveAndLoadAnEntityWithSet() {
 		assertThat(reloaded.digits).isEqualTo(new HashSet<>(asList("one", "two", "three")));
 	}
 
-	@Test //GH-1737
+	@Test // GH-1737
 	@EnabledOnFeature(SUPPORTS_ARRAYS)
 	void saveAndLoadEmbeddedArray() {
 
@@ -869,7 +868,7 @@ void saveAndLoadEmbeddedArray() {
 		assertThat(reloaded.embeddedStringList.digits).containsExactly("one", "two", "three");
 	}
 
-	@Test //GH-1737
+	@Test // GH-1737
 	@EnabledOnFeature(SUPPORTS_ARRAYS)
 	void saveAndLoadEmptyEmbeddedArray() {
 

spring-data-jdbc/src/test/java/org/springframework/data/jdbc/repository/JdbcRepositoryConcurrencyIntegrationTests.java

@@ -73,12 +73,9 @@ DummyEntityRepository dummyEntityRepository(JdbcRepositoryFactory factory) {
 		}
 	}
 
-	@Autowired
-	NamedParameterJdbcTemplate template;
-	@Autowired
-	DummyEntityRepository repository;
-	@Autowired
-	PlatformTransactionManager transactionManager;
+	@Autowired NamedParameterJdbcTemplate template;
+	@Autowired DummyEntityRepository repository;
+	@Autowired PlatformTransactionManager transactionManager;
 
 	List<DummyEntity> concurrencyEntities;
 	DummyEntity entity;
@@ -216,7 +213,7 @@ public void concurrentUpdateAndDeleteAll() throws Exception {
 	}
 
 	private void executeInParallel(CountDownLatch startLatch, CountDownLatch doneLatch,
-								   UnaryOperator<DummyEntity> deleteAction, DummyEntity entity) {
+			UnaryOperator<DummyEntity> deleteAction, DummyEntity entity) {
 		// delete
 		new Thread(() -> {
 			try {
@@ -253,13 +250,11 @@ private static DummyEntity createDummyEntity() {
 		return new DummyEntity(null, "Entity Name", new ArrayList<>());
 	}
 
-	interface DummyEntityRepository extends CrudRepository<DummyEntity, Long> {
-	}
+	interface DummyEntityRepository extends CrudRepository<DummyEntity, Long> {}
 
 	static class DummyEntity {
 
-		@Id
-		private Long id;
+		@Id private Long id;
 		String name;
 		final List<Element> content;
 
@@ -292,8 +287,7 @@ public DummyEntity withContent(List<Element> content) {
 
 	static class Element {
 
-		@Id
-		private Long id;
+		@Id private Long id;
 		final Long content;
 
 		public Element(Long id, Long content) {