-
Notifications
You must be signed in to change notification settings - Fork 37.7k
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
Avoid synchronization in AbstractAspectJAdvice#calculateArgumentBindings #26377
Comments
We had the same problem! This led to serious performance issues in our system. |
Reopened for a simpler approach as discussed in my closing comment on #28783: We should be able to drop the synchronization completely since |
That's a great idea! I'm look forward to the new version. |
Aligned with the early calculateArgumentBindings call in ReflectiveAspectJAdvisorFactory. Closes spring-projectsgh-26377
Aligned with the early calculateArgumentBindings call in ReflectiveAspectJAdvisorFactory. Closes spring-projectsgh-26377
I see
AbstractAspectJAdvice#calculateArgumentBindings
is synchronized, but if first time the Advice method calculates the argument binding, the next time it can return directly without lock, I think it can prevent some lock competition.such as:
The text was updated successfully, but these errors were encountered: