You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@wicket.apache.org by Eelco Hillenius <ee...@gmail.com> on 2007/02/08 06:02:11 UTC
maxPageVersions
Hi,
Thanks for the fix for the FilePageStore version manager Johan. Now,
there still is the UndoPageVersionManager that is used when you
configure the HttpSessionStore. The number is now max-int, which is
crazy and dangerous. I think a better max would be, say, 30.
WDYT?
Btw, I think we should apply this on 1.2.x as well or if that is too
late with the release half done, put up a warning somewhere.
Eelco
Re: maxPageVersions
Posted by Johan Compagner <jc...@gmail.com>.
no it is not
if you use the normal HttpSessionStore
Then you use the AccessStackPageMap
and that uses the Evict Strategy for (default 7) page and page versions.
So with the normal HttpSessionStore there will always be
7 unique pages or 7 page versions of one page or a combination.
johan
On 2/8/07, Eelco Hillenius <ee...@gmail.com> wrote:
>
> See also http://issues.apache.org/jira/browse/WICKET-266
>
> Eelco
>
> On 2/7/07, Eelco Hillenius <ee...@gmail.com> wrote:
> > Hi,
> >
> > Thanks for the fix for the FilePageStore version manager Johan. Now,
> > there still is the UndoPageVersionManager that is used when you
> > configure the HttpSessionStore. The number is now max-int, which is
> > crazy and dangerous. I think a better max would be, say, 30.
> >
> > WDYT?
> >
> > Btw, I think we should apply this on 1.2.x as well or if that is too
> > late with the release half done, put up a warning somewhere.
> >
> > Eelco
> >
>
Re: maxPageVersions
Posted by Eelco Hillenius <ee...@gmail.com>.
See also http://issues.apache.org/jira/browse/WICKET-266
Eelco
On 2/7/07, Eelco Hillenius <ee...@gmail.com> wrote:
> Hi,
>
> Thanks for the fix for the FilePageStore version manager Johan. Now,
> there still is the UndoPageVersionManager that is used when you
> configure the HttpSessionStore. The number is now max-int, which is
> crazy and dangerous. I think a better max would be, say, 30.
>
> WDYT?
>
> Btw, I think we should apply this on 1.2.x as well or if that is too
> late with the release half done, put up a warning somewhere.
>
> Eelco
>