[ https://issues.apache.org/jira/browse/WW-3457?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12877327#action_12877327 ]
Lukasz Lenart commented on WW-3457: ----------------------------------- Did you profile that or had any bottleneck because of that? Regards > Significant performance improvement for freemarker TagModel > ----------------------------------------------------------- > > Key: WW-3457 > URL: https://issues.apache.org/jira/browse/WW-3457 > Project: Struts 2 > Issue Type: Improvement > Affects Versions: 2.0.0 > Reporter: Jed Wesley-Smith > > The class org.apache.struts2.views.freemarker.tags.TagModel creates a new > freemarker.template.DefaultObjectWrapper for each call to > unwrapParameters(Map). The problem is that that class is quite expensive to > construct as the super-class does a quite bit of introspection to fill up its > caches. Not only is this a lot of work, it tends to hit the Introspector > class pretty hard, which internally does a lot of sychronising. In our soak > performance tests this was a significant cause of blocking in our application. > The DefaultObjectWrapper (or more properly the > freemarker.ext.beans.BeansWrapper class as that is actually all the TagModel > is using) is actually meant to be a cache that prevents exactly this kind of > slowness, so constructing a new one every time is counter-productive. > I don't have the latest source handy so I can't provide a patch, but the > change is to simply replace the line: > DefaultObjectWrapper objectWrapper = new DefaultObjectWrapper(); > with: > BeansWrapper objectWrapper = BeansWrapper.getDefaultInstance(); > in the second line of TagModel.unwrapParameters(Map params) -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.