You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@struts.apache.org by "Hudson (JIRA)" <ji...@apache.org> on 2012/12/09 22:07:21 UTC

[jira] [Commented] (WW-3766) remove struts.freemarker.mru.max.strong.size from default.properties

    [ https://issues.apache.org/jira/browse/WW-3766?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13527634#comment-13527634 ] 

Hudson commented on WW-3766:
----------------------------

Integrated in Struts2-JDK5 #17 (See [https://builds.apache.org/job/Struts2-JDK5/17/])
    WW-3766 sets struts.freemarker.mru.max.strong.size to 0 to SoftCacheStorage which performs better in heavy loaded applications (Revision 1419061)

     Result = SUCCESS
lukaszlenart : 
Files : 
* /struts/struts2/trunk/core/src/main/resources/org/apache/struts2/default.properties

                
> remove struts.freemarker.mru.max.strong.size from default.properties
> --------------------------------------------------------------------
>
>                 Key: WW-3766
>                 URL: https://issues.apache.org/jira/browse/WW-3766
>             Project: Struts 2
>          Issue Type: Improvement
>            Reporter: zhouyanming
>            Assignee: Lukasz Lenart
>            Priority: Critical
>              Labels: freemarker
>             Fix For: 2.3.8
>
>
> if struts.freemarker.mru.max.strong.size > 0 ,freemarker will use MruCacheStorage instead of SoftCacheStorage,and MruCacheStorage is not concurrent,it will synchronized on every call,it will affect performance. 
> in TemplateCache:
> {code:java}
> CachedTemplate cachedTemplate;
> if(isStorageConcurrent) {
>     cachedTemplate = (CachedTemplate)storage.get(tk);
> }
> else {
>     synchronized(storage) {
>        cachedTemplate = (CachedTemplate)storage.get(tk);
>     }
> }
> {code}
> please remove struts.freemarker.mru.max.strong.size from default.properties lets developers make their own choice.
> a better solution is implements a concurrent MruCacheStorage for struts2

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira