i want use @EnableCaching(mode = AdviceMode.ASPECTJ) but it not work?
Comment From: philwebb
There's not enough information here to diagnose if this is bug or not. Please provide a small sample application that shows your problem.
Comment From: brucelwl
@philwebb this is my demo code. https://github.com/brucelwl/demo
before run the main class DemoApplication, you need add the JVM options -javaagent:path\spring-instrument-5.1.8.RELEASE.jar. The console log will print 'select not use cache' twice, but if the @EnableCaching(mode = AdviceMode.ASPECTJ) takes effect, it only needs to be printed once.
Comment From: snicoll
@brucelwl thank you for the sample. It does not work either when I replace the use of Spring Boot with plain Spring framework code.
Comment From: philwebb
@brucelwl We've transferred the issue to Spring Framework for that team to take a look at.
Comment From: brucelwl
@sbrannen Please confirm this issue. you may need to roll back to the corresponding version of cache-aspect, f5db13a5d7cb50ee6b173cc1f7cd8c76dbad5e80
Comment From: philwebb
@brucelwl I can't find a commit for f5db13a5d7cb50ee6b173cc1f7cd8c76dbad5e80, please can you provide a link so we know what you are referring to.
Comment From: brucelwl
@philwebb What i am referring to is https://github.com/brucelwl/demo, I used this project to submit other issue, but the demo for issues#23389 https://github.com/spring-projects/spring-framework/issues/23389 needs to be rolled back to the f5db13a5d7cb50ee6b173cc1f7cd8c76dbad5e80 version, You've already answered https://github.com/spring-projects/spring-framework/issues/23389#issuecomment-516801570, but no confirmation information has been received.
Comment From: philwebb
@brucelwl It's best if you create one sample per issue that you raise. To clarify, you're saying that this commit needs to be reverted in order to trigger the issue?
no confirmation information has been received
There's a lot of work on at the moment so it may take some time for someone from the framework team to triage the issue.
Comment From: brucelwl
@philwebb Okay, I'll create branches for different issues later to demonstrate the problem.
Comment From: snicoll
Closing due to the lack of feedback.