Author: ssmiweve
Date: 2008-11-16 23:02:30 +0100 (Sun, 16 Nov 2008)
New Revision: 6939

Modified:
   trunk/war/src/main/java/no/sesat/search/http/filters/SiteJspLoaderFilter.java
Log:
performance tracing and a small performance fix (one less anonymous class)

Modified: 
trunk/war/src/main/java/no/sesat/search/http/filters/SiteJspLoaderFilter.java
===================================================================
--- 
trunk/war/src/main/java/no/sesat/search/http/filters/SiteJspLoaderFilter.java   
    2008-11-16 22:01:03 UTC (rev 6938)
+++ 
trunk/war/src/main/java/no/sesat/search/http/filters/SiteJspLoaderFilter.java   
    2008-11-16 22:02:30 UTC (rev 6939)
@@ -34,9 +34,9 @@
 import javax.servlet.ServletResponse;
 import javax.servlet.http.HttpServletRequest;
 import no.sesat.search.site.Site;
-import no.sesat.search.site.SiteContext;
 import no.sesat.search.site.config.BytecodeLoader;
 import no.sesat.search.site.config.UrlResourceLoader;
+import org.apache.commons.lang.time.StopWatch;
 import org.apache.log4j.Logger;
 
 
@@ -157,6 +157,9 @@
             final HttpServletRequest request,
             final String jsp) throws MalformedURLException{
 
+        final StopWatch stopWatch = new StopWatch();
+        stopWatch.start();
+
         byte[] golden = new byte[0];
 
         // search skins for the jsp and write it out to "golden"
@@ -171,11 +174,7 @@
 
             final Site finalSite = site;
             final BytecodeLoader bcLoader = 
UrlResourceLoader.newBytecodeLoader(
-                    new SiteContext(){
-                        public Site getSite() {
-                            return finalSite;
-                        }
-                    },
+                    finalSite.getSiteContext(),
                     jsp,
                     null
             );
@@ -238,6 +237,9 @@
                 LOG.error(ex.getMessage(), ex);
             }
         }
+
+        stopWatch.stop();
+        LOG.trace("SiteJspLoaderFilter.downloadJsp(..) took " + stopWatch);
     }
 
     //// Imported from org.catalina.jasper.Constants

_______________________________________________
Kernel-commits mailing list
[email protected]
http://sesat.no/mailman/listinfo/kernel-commits

Reply via email to