Author: markt
Date: Sat Sep 22 19:54:15 2012
New Revision: 1388880
URL: http://svn.apache.org/viewvc?rev=1388880&view=rev
Log:
Minor performance tweak
Modified:
tomcat/trunk/java/org/apache/catalina/core/StandardHostValve.java
Modified: tomcat/trunk/java/org/apache/catalina/core/StandardHostValve.java
URL:
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/core/StandardHostValve.java?rev=1388880&r1=1388879&r2=1388880&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/core/StandardHostValve.java (original)
+++ tomcat/trunk/java/org/apache/catalina/core/StandardHostValve.java Sat Sep
22 19:54:15 2012
@@ -60,6 +60,12 @@ final class StandardHostValve extends Va
private static final Log log = LogFactory.getLog(StandardHostValve.class);
+ // Saves a call to getClassLoader() on very request. Under high load these
+ // calls took just long enough to appear as a hot spot (although a very
+ // minor one) in a profiler.
+ private static final ClassLoader MY_CLASSLOADER =
+ StandardHostValve.class.getClassLoader();
+
protected static final boolean STRICT_SERVLET_COMPLIANCE;
protected static final boolean ACCESS_SESSION;
@@ -184,12 +190,10 @@ final class StandardHostValve extends Va
// Restore the context classloader
if (Globals.IS_SECURITY_ENABLED) {
- PrivilegedAction<Void> pa = new PrivilegedSetTccl(
- StandardHostValve.class.getClassLoader());
+ PrivilegedAction<Void> pa = new PrivilegedSetTccl(MY_CLASSLOADER);
AccessController.doPrivileged(pa);
} else {
- Thread.currentThread().setContextClassLoader
- (StandardHostValve.class.getClassLoader());
+ Thread.currentThread().setContextClassLoader(MY_CLASSLOADER);
}
}
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org
