Author: remy.maucherat(a)jboss.com Date: 2011-10-28 11:29:40 -0400 (Fri, 28 Oct 2011) New Revision: 1861 Modified: trunk/java/org/apache/jasper/runtime/PerThreadTagHandlerPool.java trunk/java/org/apache/jasper/runtime/TagHandlerPool.java Log: - Port performance patch by dropping not so useful logging. I don't see how it really changes things for web, since it won't unload JSPs. Modified: trunk/java/org/apache/jasper/runtime/PerThreadTagHandlerPool.java =================================================================== --- trunk/java/org/apache/jasper/runtime/PerThreadTagHandlerPool.java 2011-10-28 09:26:16 UTC (rev 1860) +++ trunk/java/org/apache/jasper/runtime/PerThreadTagHandlerPool.java 2011-10-28 15:29:40 UTC (rev 1861) @@ -136,15 +136,13 @@ try { ptd.handlers[i].release(); } catch (Exception e) { - log.warn("Error processing release on tag instance of " - + ptd.handlers[i].getClass().getName(), e); + // Ignore } if (Constants.INJECT_TAGS || Constants.USE_INSTANCE_MANAGER_FOR_TAGS) { try { instanceManager.destroyInstance(ptd.handlers[i]); } catch (Exception e) { - log.warn("Error processing preDestroy on tag instance of " - + ptd.handlers[i].getClass().getName(), e); + // Ignore } } } Modified: trunk/java/org/apache/jasper/runtime/TagHandlerPool.java =================================================================== --- trunk/java/org/apache/jasper/runtime/TagHandlerPool.java 2011-10-28 09:26:16 UTC (rev 1860) +++ trunk/java/org/apache/jasper/runtime/TagHandlerPool.java 2011-10-28 15:29:40 UTC (rev 1861) @@ -37,8 +37,6 @@ public static final String OPTION_TAGPOOL="tagpoolClassName"; public static final String OPTION_MAXSIZE="tagpoolMaxSize"; - protected Logger log = Logger.getLogger(TagHandlerPool.class); - // index of next available tag handler private int current; protected InstanceManager instanceManager = null; @@ -163,8 +161,7 @@ try { instanceManager.destroyInstance(handlers[i]); } catch (Exception e) { - log.warn("Error processing preDestroy on tag instance of " - + handlers[i].getClass().getName(), e); + // Ignore } } }