Ignore update count without version.

When an update results in 0 updated rows we now ignore it, instead of throwing an exception.

The reason is that at least some databases return 0 when a row for updating was found, but none of the values changed.

Closes #2176

Author: schauder

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

@@ -348,12 +348,7 @@ private <T> RelationalPersistentEntity<T> getRequiredPersistentEntity(Class<T> t
 	}
 
 	private <T> void updateWithoutVersion(DbAction.UpdateRoot<T> update) {
-
-		if (!accessStrategy.update(update.entity(), update.getEntityType())) {
-
-			throw new IncorrectUpdateSemanticsDataAccessException(
-					String.format(UPDATE_FAILED, update.entity(), getIdFrom(update)));
-		}
+		accessStrategy.update(update.entity(), update.getEntityType());
 	}
 
 	private <T> void updateWithVersion(DbAction.UpdateRoot<T> update) {

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

@@ -579,8 +579,7 @@ void updateFailedRootDoesNotExist() {
 		LegoSet entity = new LegoSet();
 		entity.id = 100L; // does not exist in the database
 
-		assertThatExceptionOfType(IncorrectUpdateSemanticsDataAccessException.class) //
-				.isThrownBy(() -> template.save(entity));
+		assertThatCode(() -> template.save(entity)).doesNotThrowAnyException();
 	}
 
 	@Test // DATAJDBC-112
@@ -1180,7 +1179,7 @@ void saveAndUpdateAggregateWithIdAndNullVersion() {
 		aggregate.setVersion(null);
 		aggregate.setId(23L);
 
-		assertThatThrownBy(() -> template.save(aggregate)).isInstanceOf(IncorrectUpdateSemanticsDataAccessException.class);
+		assertThatCode(() -> template.save(aggregate)).doesNotThrowAnyException();
 	}
 
 	@Test // DATAJDBC-462