@Cacheable not working on interface method in Spring Boot 1.4.2+

[ahmednasir91]

We use Spring cache annotation

@Cacheable(cacheManager = "cacheManager", value = "cache-test", key = "'test-key2'")

This used to work well till Spring Boot 1.4.1 and the interface methods are annotated, since after upgrading to Spring Boot 1.4.3 & also tried the same on 1.4.4 the interface annotated methods have stopped working.

After digging deeper, we found it stops working with Spring Data JPA

<dependency>
	<groupId>org.springframework.boot</groupId>
	<artifactId>spring-boot-starter-data-jpa</artifactId>
</dependency>

If you exclude this repo, it starts working on interface methods.

This is the sample repo, to reproduce the issue https://github.com/ahmednasir91/spring-cache-test
The project has 3 branches

• master SB v1.4.4 & Data JPA commented - Caching works on interface methods
• cache-not-working SB 1.4.4, Data JPA enabled - Caching doesn't work for interface method
• working-boot-1.4.1 SB 1.4.1 Data JPA enabled - Caching works on interface methods

Following are the APIs

These two have caching on interface
GET /test/hello2 
GET /test/hello

This one has on implementation
GET /test/hello-list

[snicoll]

Thanks for the sample but I've built something that is more reasonable based on yours (you really do not need a redis store to reproduce this issue!)

https://github.com/snicoll-scratches/test-cache-interface

The mix between annotations on interface and concrete classes is not something we recommend doing but I can confirm that adding the JPA starter is enough to trigger the problem. It works with Spring Boot 1.4.1..RELEASE (Spring Framework 4.3.3.RELEASE) and does not anymore as of 1.4.2.RELEASE (Spring Framework 4.3.4.RELEASE).

If I upgrade the spring framework version to use with 1.4.1.RELEASE with 4.3.4.RELEASE I am able to reproduce the problem. This looks like a framework regression but I have no idea why data jpa is triggering it.

[ahmednasir91]

Thanks @snicoll

We are using @Cacheable on interfaces only, just for sample i added to both interfaces & concrete class.

[snicoll]

Ah that makes sense, thanks for clarifying.

[snicoll]

After further investigation, this has been confirmed as a regression in Spring Framework, watch SPR-15271 for updates. I am closing this one in favour of #8248