Fix method return for delete execution returning primitive numbers.

We now properly check for assignability of numeric values considering primitive types.

Closes #4015

Author: mp911de

spring-data-jpa/src/main/java/org/springframework/data/jpa/repository/query/JpaQueryExecution.java

@@ -296,9 +296,10 @@ public DeleteExecution(EntityManager em) {
 
 			Query query = jpaQuery.createQuery(accessor);
 			List<?> resultList = query.getResultList();
+			Class<?> returnType = jpaQuery.getQueryMethod().getReturnType();
 
-			boolean simpleBatch = Number.class.isAssignableFrom(jpaQuery.getQueryMethod().getReturnType())
-					|| org.springframework.data.util.ReflectionUtils.isVoid(jpaQuery.getQueryMethod().getReturnType());
+			boolean simpleBatch = ClassUtils.isAssignable(Number.class, returnType)
+					|| org.springframework.data.util.ReflectionUtils.isVoid(returnType);
 			boolean collectionQuery = jpaQuery.getQueryMethod().isCollectionQuery();
 
 			if (!simpleBatch && !collectionQuery) {

spring-data-jpa/src/test/java/org/springframework/data/jpa/repository/UserRepositoryTests.java

@@ -1598,12 +1598,21 @@ void deleteByShouldRemoveElementsMatchingDerivedQuery() {
 		assertThat(repository.countByLastname(firstUser.getLastname())).isZero();
 	}
 
-	@Test // DATAJPA-460
+	@Test // DATAJPA-460, GH-4015
 	void deleteByShouldReturnNumberOfEntitiesRemovedIfReturnTypeIsLong() {
 
 		flushTestUsers();
 
 		assertThat(repository.removeByLastname(firstUser.getLastname())).isOne();
+		assertThat(repository.removeOneByLastname(secondUser.getLastname())).isOne();
+	}
+
+	@Test // GH-4015
+	void deleteByShouldReturnNumberOfEntitiesRemovedIfReturnTypeIsInt() {
+
+		flushTestUsers();
+
+		assertThat(repository.removeOneMoreByLastname(secondUser.getLastname())).isOne();
 	}
 
 	@Test // DATAJPA-460

spring-data-jpa/src/test/java/org/springframework/data/jpa/repository/sample/UserRepository.java

@@ -296,6 +296,10 @@ Window<User> findTop3ByFirstnameStartingWithOrderByFirstnameAscEmailAddressAsc(S
 	// DATAJPA-460
 	Long removeByLastname(String lastname);
 
+	long removeOneByLastname(String lastname);
+
+	int removeOneMoreByLastname(String lastname);
+
 	// DATAJPA-460
 	List<User> deleteByLastname(String lastname);