-
Notifications
You must be signed in to change notification settings - Fork 38k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Issue with @Async in version 3.1.2 linking to the task executor it is assigned to (instead of the default) [SPR-9575] #14209
Comments
Chris Beams commented Thanks, Ken. Looking forward to the repro issue. |
Ken Young commented Learning Git (slowly), so bear with me. Need to minimize my MVC test project. |
Chris Beams commented No rush; thanks for taking the time. |
Ken Young commented I have added the pull request for this project to the GitHub repo. Thanks |
Sam Brannen commented Pull request: spring-attic/spring-framework-issues#29 |
Chris Beams commented Hi Ken, Take a look at spring-attic/spring-framework-issues@2a8b821#L0R36 That statement prints Take a look at and play around with this simple test case that I contributed. You'll see there that executor selection does indeed work: spring-attic/spring-framework-issues@de91e04 If you're able to fix things up so that you've actually got an async proxy and it still fails to respect the #value attribute of the Thanks |
Ken Young commented Chris, The issue was that class that has the async method annotated as follows:
Once this was added, the annotation worked flawlessly. Thanks |
Ittai Zeidman commented
Thanks, |
Ken Young commented The annotation that I was missing was
I used an interface, but I put the annotation at the implementation level. |
Ittai Zeidman commented Thanks Ken! |
Ken Young opened SPR-9575 and commented
Issue with
@Async
in version 3.1.2 linking to the task executor it is assigned to (instead of the default)Affects: 3.1.2
Issue Links:
@Async
@Async
"0 votes, 5 watchers
The text was updated successfully, but these errors were encountered: