You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by bu...@apache.org on 2012/03/30 12:15:35 UTC
DO NOT REPLY [Bug 52998] Performance issue with
ExpressionFactory.newInstance()
https://issues.apache.org/bugzilla/show_bug.cgi?id=52998
--- Comment #1 from Konstantin Kolinko <kn...@gmail.com> 2012-03-30 10:15:35 UTC ---
Created attachment 28518
--> https://issues.apache.org/bugzilla/attachment.cgi?id=28518
2012-03-30_tc8_52998_ExpressionFactory.patch
Patch against trunk that implements caching in ExpressionFactory.
Notes:
1. r1307310 should be ported to 7.0.x before this patch.
Maybe whitespace cleanup r1187778 shall be ported as well.
2. This patch only implements the caching.
The issue with static references mentioned in Comment 0 is not addressed.
3. There is no way to clean stale cache entries from stopped webapps,
but the cache entry objects are small, so I think we can live with it.
Weak references are used, so the class loaders from undeployed apps should be
eligible for garbage collection.
--
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org