Author: markt
Date: Tue Nov 16 18:07:56 2010
New Revision: 1035720
URL: http://svn.apache.org/viewvc?rev=1035720&view=rev
Log:
Session manager performance
Move sync. No performance change but sync requirements for sub-classes are
clearer.
Still looking to remove sync completely.
Modified:
tomcat/trunk/java/org/apache/catalina/session/ManagerBase.java
Modified: tomcat/trunk/java/org/apache/catalina/session/ManagerBase.java
URL:
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/session/ManagerBase.java?rev=1035720&r1=1035719&r2=1035720&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/session/ManagerBase.java (original)
+++ tomcat/trunk/java/org/apache/catalina/session/ManagerBase.java Tue Nov 16
18:07:56 2010
@@ -959,7 +959,7 @@ public abstract class ManagerBase extend
}
- protected void getRandomBytes(byte bytes[]) {
+ protected synchronized void getRandomBytes(byte bytes[]) {
// Generate a byte array containing a session identifier
if (devRandomSourceIsValid && randomIS == null) {
setRandomFile(devRandomSource);
@@ -1008,9 +1008,7 @@ public abstract class ManagerBase extend
}
while (resultLenBytes < this.sessionIdLength) {
- synchronized (this) {
- getRandomBytes(random);
- }
+ getRandomBytes(random);
MessageDigest md = digests.poll();
if (md == null) {
// If this fails, NPEs will follow. This should never fail
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org
