Trivial Description
I've already described this idea in WICKET-2889. The proposed implementation (patch attached to WICKET-2889) was rejected though:
> From: Johan Compagner <jcompagner@gmail.com>
> Date: Mon, 9 Aug 2010 17:00:12 +0200
> Message-ID: <AANLkTim3dBcyouucpA+Lqj2mUnz1zCofWx8D==ZMiQcr@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%)
Attachments
Issue Links
- is part of
-
WICKET-2889 Remove reference to last page from idle SecondLevelCachePageMap
-
- Closed
-