You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by "Martin Grigorov (JIRA)" <ji...@apache.org> on 2011/09/20 14:19:09 UTC

[jira] [Resolved] (WICKET-3091) Remove references to last page from SecondLevelCacheSessionStore from idle sessions to reduce memory usage

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

Martin Grigorov resolved WICKET-3091.
-------------------------------------

       Resolution: Fixed
    Fix Version/s: 1.5.0

The page storages for 1.5.0 have better configuration settings. Check https://cwiki.apache.org/confluence/x/qIaoAQ and IStoreSettings class.

> Remove references to last page from SecondLevelCacheSessionStore from idle sessions to reduce memory usage
> ----------------------------------------------------------------------------------------------------------
>
>                 Key: WICKET-3091
>                 URL: https://issues.apache.org/jira/browse/WICKET-3091
>             Project: Wicket
>          Issue Type: Improvement
>          Components: wicket
>    Affects Versions: 1.4.12
>            Reporter: Stefan Fussenegger
>            Priority: Trivial
>             Fix For: 1.5.0
>
>
> I've already described this idea in WICKET-2889. The proposed implementation (patch attached to WICKET-2889) was rejected though:
> > From: Johan Compagner <jc...@gmail.com> 
> > Date: Mon, 9 Aug 2010 17:00:12 +0200 
> > Message-ID: <AA...@mail.gmail.com> 
> > Subject: Re: Could somebody please comment on WICKET-2889? 
> > To: users@wicket.apache.org 
> > Content-Type: text/plain; charset=ISO-8859-1 
> > 
> > I am against that patch. 
> > You keep pagemaps in memory in the applicaiton context! 
> > Those are session stuff stored in the HttpSession. You shouldnt keep 
> > reference to those stuff. 
> > This can break all kind of things (for example clustering) 
> > 
> > If you want something like that, then it is fine if we need to change 
> > something so that you can overwrite some stuff so that you can do what 
> > you want without patching wicket. 
> > But i dont want this to be default in wicket. 
> As I understood, the objects are to the implementation not the idea itself. I still consider this a very valuable improvement as it has the potential to drastically reduce Wicket's memory consumption in productions environments, especially for applications that make (heavy) use of statefull pages. An average page in our application used 115K of memory. With 1500 live sessions within 30 minutes, last pages accumulated for 170M memory usage. By expiring the last page after 10 minutes, this could be cut by 2/3 or 113M. (btw: if I remember correctly, Wicket's heap usage wasn't more than 220M in total, hence reducing overall memory usage by about 50%)

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira