Re: Lock timeout per page class

2014-10-28 Thread Martin Grigorov
Hi, I share Sebastien's concern. I'll see how to workaround this. Martin Grigorov Wicket Training and Consulting https://twitter.com/mtgrigorov On Sat, Oct 25, 2014 at 2:57 PM, Sebastien seb...@gmail.com wrote: Hi Guillaume, Generally speaking, you cannot call a non final method from a

Re: Lock timeout per page class

2014-10-28 Thread Martin Grigorov
Here is my version: http://pastie.org/9680245 Please create a ticket in JIRA if you like it. Martin Grigorov Wicket Training and Consulting https://twitter.com/mtgrigorov On Tue, Oct 28, 2014 at 9:52 AM, Martin Grigorov mgrigo...@apache.org wrote: Hi, I share Sebastien's concern. I'll see

Re: Lock timeout per page class

2014-10-28 Thread Guillaume Smet
Hi Martin, Looks sane to me. I created a JIRA issue: https://issues.apache.org/jira/browse/WICKET-5740 . Thanks again for your help! On Tue, Oct 28, 2014 at 9:45 AM, Martin Grigorov mgrigo...@apache.org wrote: Here is my version: http://pastie.org/9680245 Please create a ticket in JIRA if you

Re: Lock timeout per page class

2014-10-25 Thread Guillaume Smet
Hi Martin, I got something working with the following changes in Wicket: https://github.com/openwide-java/wicket/commit/6374a4a7c6fb66841143a88933523f97305cf1a4 Do you consider this commitable? If so, I can create a JIRA issue and push a PR. Having the pageId in the getTimeout call is quite

Re: Lock timeout per page class

2014-10-25 Thread Sebastien
Hi Guillaume, Generally speaking, you cannot call a non final method from a constructor... Best regards, Sebastien On Oct 25, 2014 1:32 PM, Guillaume Smet guillaume.s...@gmail.com wrote: Hi Martin, I got something working with the following changes in Wicket:

Re: Lock timeout per page class

2014-10-24 Thread Martin Grigorov
Hi Guillaume, Sorry for not thinking more carefully about this the first time! I'm afraid it is not possible to do it the way I suggested. PageAccessSynchronizer is the entry point to start using a page and it works only with pageId! Here is a new hackish approach: Store pageId - pageClassName

Re: Lock timeout per page class

2014-10-24 Thread Guillaume Smet
Hi Martin, Yeah, I thought about that too but I'm not sure of the best place to build the pageId - pageClassName map. Any advice about this? Once I'll get this working, I'll build a PR for the few changes I made in Wicket (based on what you proposed earlier). Would be nice to have them in 6.18.

Re: Lock timeout per page class

2014-10-24 Thread Martin Grigorov
If you have a base page then BasePage#onInitialize() should be a good place. Or you could add the pageIds of the special/slow pages only in the map. Otherwise you may use PageRequestHandlerTracker#getLastHandler in a custom IRequestCycleListener#onDetach(). Martin Grigorov Wicket Training and

Re: Lock timeout per page class

2014-10-23 Thread Guillaume Smet
Hi Martin, On Wed, Oct 22, 2014 at 9:51 AM, Martin Grigorov mgrigo...@apache.org wrote: I'd like to avoid moving the logic that gets the timeout from Session.PageAccessSynchronizerProvider to PageAccessSynchronizer because this way it will use Application.get() everytime and most apps don't

Re: Lock timeout per page class

2014-10-22 Thread Martin Grigorov
Hi, You are right! On Tue, Oct 21, 2014 at 11:11 PM, Guillaume Smet guillaume.s...@gmail.com wrote: Hi Martin, On Tue, Oct 21, 2014 at 1:19 PM, Martin Grigorov mgrigo...@apache.org wrote: You can use org.apache.wicket.request.cycle.PageRequestHandlerTracker#getFirstHandler() to check

Lock timeout per page class

2014-10-21 Thread Guillaume Smet
Hi, We have a few pages in our application which might take a long time to generate. This is definitely not the usual case but there are a few of them. Thus we were forced to define a high lockTimeout to be sure these pages can be served. The fact is that we would really like to have a far

Re: Lock timeout per page class

2014-10-21 Thread Martin Grigorov
Hi Guillaume, You can use org.apache.wicket.request.cycle.PageRequestHandlerTracker#getFirstHandler() to check what is the requested page in your own org.apache.wicket.settings.def.RequestCycleSettings#getTimeout Martin Grigorov Wicket Training and Consulting https://twitter.com/mtgrigorov On

Re: Lock timeout per page class

2014-10-21 Thread Guillaume Smet
Thanks Martin! We will give it a try and post what we got. -- Guillaume On Tue, Oct 21, 2014 at 1:19 PM, Martin Grigorov mgrigo...@apache.org wrote: Hi Guillaume, You can use org.apache.wicket.request.cycle.PageRequestHandlerTracker#getFirstHandler() to check what is the requested page

Re: Lock timeout per page class

2014-10-21 Thread Guillaume Smet
Hi Martin, On Tue, Oct 21, 2014 at 1:19 PM, Martin Grigorov mgrigo...@apache.org wrote: You can use org.apache.wicket.request.cycle.PageRequestHandlerTracker#getFirstHandler() to check what is the requested page in your own org.apache.wicket.settings.def.RequestCycleSettings#getTimeout