Session is fine IMHO. -Matej
On Feb 12, 2008 7:53 PM, Igor Vaynberg <[EMAIL PROTECTED]> wrote: > hmm, putting said counter into session should work well, and would > result in almost no syncing... > > -igor > > > > On Feb 12, 2008 10:52 AM, Igor Vaynberg <[EMAIL PROTECTED]> wrote: > > no, requestcycle wont work. markup ids have to be unique across > > requests - for the lifecycle of the page. > > > > clustering is also something i didnt think about... > > > > -igor > > > > > > > > On Feb 12, 2008 10:49 AM, Martijn Dashorst <[EMAIL PROTECTED]> wrote: > > > The request cycle sounds like a good place to put such a counter. Then the > > > counter is request relative, and synchronized by default for the request. > > > Martijn > > > > > > > > > On 2/12/08, Ryan Sonnek <[EMAIL PROTECTED]> wrote: > > > > > > > > I'm definitely not against the idea, but I would caution introducing > > > > an application wide ID generator. This would introduce a *single* > > > > synchronization point for component creation which *could* introduce a > > > > performance bottleneck. > > > > > > > > Only way to find out is to try, but this is just a word of caution. =) > > > > > > > > On Feb 12, 2008 12:42 PM, Igor Vaynberg <[EMAIL PROTECTED]> wrote: > > > > > > > > > > one of the nagging things on the lists is that markup id is not > > > > > available until render time, this gets a lot of users hung up. the > > > > > problem is that we use a counter inside the page to generate the > > > > > unique id. what if instead we use a static counter, that will make > > > > > markup ids unique across pages also. the only problem is that it will > > > > > make the id counter increment faster, but with a Long we wont run out > > > > > any time soon. we should also keep the id string as short as possible, > > > > > maybe encode the integer using [0-9a-zA-Z] instead of just [0-9], or > > > > > even to make it faster encode it into hex... > > > > > > > > > > if no objections are raised i will make the change...will be nice to > > > > > have a major improvement to go along with 1.3.2 anyways > > > > > > > > > > -igor > > > > > > > > > > > > > > > > > > > > > -- > > > Buy Wicket in Action: http://manning.com/dashorst > > > Apache Wicket 1.3.1 is released > > > Get it now: http://www.apache.org/dyn/closer.cgi/wicket/1.3.1 > > > > > > -- Resizable and reorderable grid components. http://www.inmethod.com
