Changes in CVS and performance hints

[Santiago Gala]

I have committed a few changes in cvs:

- Patch in JetspeedURLManagerService.java to have it saving correctly 
urls containing "," (spoiled the persistency of "bad" url from time to time)

- Patch in init() of VariableResourcesService.java so that it waits to 
be properly initialized. Juan Carlos was having problems with this one.

- The patches I promised last thursday Map ---> Dictionary

- A patch in URIEncoder.java and StringUtils.java This one is important 
for performance.

The code there was creating something like 6/10 StringBuffers and a 
couple of Strings for each call to URIEncoder.encode( url ) or 
URIEncoder.decode( url ).

When I did HEAP profiling, it showed as one of the "easy" strongest 
hotspots for memory allocation. I cleaned it, and it works faster now 
(it should be noticeable, specially together with next hint).

- Another hotspot I found was related to tomcat, and due to the fact 
that the default for webapps is "reloadable". This makes tomcat work a 
lot in each request, trying to know if the jar has changed. To solve, 
add something similar to:

        <Context path="/jetspeed"
                  docBase="webapps/jetspeed"
                  crossContext="false"
                  debug="0"
                  reloadable="false"
                  trusted="false" >
         </Context>

In conf/server.xml

Again, it will give faster operations, but the servlets in this context 
will not reload when recharged. Reloading, in any case, did not work, as
it gave an error related with a ClassCast of a CapabilityMap 
(classloader issues).

If you look at my working copy, you may notice the performance should be 
better, except when there are bandwidth issues.



--
--------------------------------------------------------------
To subscribe:        [EMAIL PROTECTED]
To unsubscribe:      [EMAIL PROTECTED]
Search: <http://www.mail-archive.com/jetspeed@list.working-dogs.com/>
List Help?:          [EMAIL PROTECTED]