You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@struts.apache.org by "Musachy Barroso (JIRA)" <ji...@apache.org> on 2009/07/21 19:17:59 UTC

[jira] Resolved: (WW-3202) UtilTimerStack activation (XWork)

     [ https://issues.apache.org/struts/browse/WW-3202?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Musachy Barroso resolved WW-3202.
---------------------------------

       Resolution: Fixed
    Fix Version/s:     (was: Future)
                   2.1.8

fixed in xwork trunk

> UtilTimerStack activation (XWork)
> ---------------------------------
>
>                 Key: WW-3202
>                 URL: https://issues.apache.org/struts/browse/WW-3202
>             Project: Struts 2
>          Issue Type: Sub-task
>            Reporter: Musachy Barroso
>            Assignee: Musachy Barroso
>            Priority: Minor
>             Fix For: 2.1.8
>
>
> On every UtilTimerStack push/pop, UtilTimerStack  checks if it is active or not. isActive checks if a system property which is implemented with a HashTable.  Some traces:
> Method Traces	Count	Total Latency(ms)
> java.util.Hashtable.get(Object)	33	1,151.000
>    java.util.Properties.getProperty(String)	33	1,151.000
>       java.lang.System.getProperty(String)	33	1,151.000
>          com.opensymphony.xwork2.util.profiling.UtilTimerStack.isActive()	32	1,115.000
>             com.opensymphony.xwork2.util.profiling.UtilTimerStack.push(String)	23	807.000
>                org.apache.struts2.dispatcher.Dispatcher.serviceAction(HttpServletRequest, HttpServletResponse, ServletContext, ActionMapping)	14	518.000
>                com.opensymphony.xwork2.DefaultActionInvocation.invoke()	7	215.000
>                com.opensymphony.xwork2.DefaultActionInvocation.executeResult()	1	50.000
>                com.opensymphony.xwork2.DefaultActionInvocation.invokeAction(Object, ActionConfig)	1	23.000
>             com.opensymphony.xwork2.util.profiling.UtilTimerStack.pop(String)	9	307.000
>          sun.security.action.GetPropertyAction.run()	1	36.000
>             sun.security.action.GetPropertyAction.run()	1	36.000
> I think checking for the system property should be done only once, on a static init. The profiler util can be activated at runtime using it's API or a parameter.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.