Missing @Transactional on SimpleJpaRepository.delete(spec)

[gbrehmer]

Is there a reason that this delete method variant has no predefined @Transactional marker?

spring-data-jpa/spring-data-jpa/src/main/java/org/springframework/data/jpa/repository/support/SimpleJpaRepository.java

Line 479 in 4387a31

public long delete(Specification<T> spec) {

Currently this breaks support (or needs additional boilerplate code) for the best practice of adding @Transactional(readOnly=true) at repository class level

[mp911de]

That's an oversight on our end, the method should be marked @Transactional.

[quaff]

@mp911de Please see #3194 to decide if we need a test case to prevent such oversight.