Thanks Sven! That looks much better. I'll give it a try as soon as I can.
Best regards, Thomas On Fri, Mar 27, 2020 at 2:23 PM Sven Meier <s...@meiers.net> wrote: > Hi Thomas, > > your question comes at the right time. > > I was able to improve the implementation with a new CachingPageStore: > > > https://github.com/apache/wicket/blob/8df3528dc44a08b7d375c20e764a3664cd6a5f30/wicket-core/src/main/java/org/apache/wicket/DefaultPageManagerProvider.java#L145 > > You can now use InMemoryPageStore as a cache too. > > Have fun > Sven > > > On 27.03.20 09:34, Sven Meier wrote: > > Hi Thomas, > > > > I thought I covered that usecase, but I will have to take a look. > > > > Thanks for testing Wicket 9 > > Sven > > > > On 25.03.20 20:10, Thomas Heigl wrote: > >> Maybe the same approach could be used as for InSessionPageStore that > >> can be > >> used as cache and a store: > >> > >> > https://github.com/apache/wicket/commit/894799e01227781be76886b2d1cdb2a424c812e0 > >> > >> > >> On Wed, Mar 25, 2020 at 6:35 PM Thomas Heigl <tho...@umschalt.com> > >> wrote: > >> > >>> Hi all, > >>> > >>> I just merged our master in our Wicket 9 branch and I ran into an > >>> issue: > >>> > >>> Our current configuration with Wicket 8 looks like this: > >>> > >>> PageStore = PerSessionPageStore > >>> DataStore = RedisDataStore > >>> > >>> So the page store keeps the last couple of pages of a session in memory > >>> and Redis is used as a persistent store. > >>> > >>> I tried to recreate this behavior with Wicket 9: > >>> > >>> SessionStore = InMemoryPageStore > >>> PersistentStore = RedisDataStore > >>> > >>> This looks correct, but it *does not work* because InMemoryPage store > >>> implements AbstractPersistentPageStore and does *not* delegate to the > >>> next store in the chain. > >>> > >>> So we basically lost the option to use a memory page store in front > >>> of a > >>> persistent store. > >>> > >>> We need this functionality because we are using Spring Session and > >>> cannot > >>> use the session as a page store. > >>> > >>> Would it be possible to add an InMemory store that delegates to the > >>> next > >>> store in the chain? Or do I have to implement it myself? > >>> > >>> Best regards, > >>> > >>> Thomas > >>> > > > > --------------------------------------------------------------------- > > To unsubscribe, e-mail: users-unsubscr...@wicket.apache.org > > For additional commands, e-mail: users-h...@wicket.apache.org > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: users-unsubscr...@wicket.apache.org > For additional commands, e-mail: users-h...@wicket.apache.org > >
