Skip to content
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

Nicholas Irving ([email protected]) #22

Open
wants to merge 1 commit into
base: 2.1-master
Choose a base branch
from

Conversation

nirving
Copy link

@nirving nirving commented Oct 19, 2014

  • Fixed an issue where the ClassLoader was being hit heavily to check to see what Spring Dialect was being used. Now check from one class instead of the ClassLoader. This improves a lock found in WebLogic 10.3.6, even after using a Filtered ClassLoader.

Instead of checking if what classes are on the ClassPath everytime with Class.forName, this change caches the class lookup for each Spring Dialect. I have seen similair changes in the code base to cache configuration items, but not the class. This improveda lock found in WebLogic when executing over 20+ requests per second.

* Fixed an issue where the ClassLoader was being hit heavily to check to see what Spring Dialect was being used. Now check from one class instead of the ClassLoader. This improves a lock found in WebLogic 10.3.6, even after using a Filtered ClassLoader.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant