Just curious: in 5527-inefficient-DefaultDataStore branch (8908b8f2) you are now using a ConcurrentLinkedDeque as a cache. Why not use a map here, e.g. ConcurrentSkipListMap with entries ordered by a timestamp or whatever?
Plus: why is that cache global in terms of that it contains all pages of all sessions? Couldn’t we have a cache that stores pages per sessionId? Asking because I’m currently implementing an own IPageStore and want to learn about the mechanics and possible pitfalls. Cheers, -Tom On 06.03.2014, at 15:01, Martin Grigorov <[email protected]> wrote: > In the meantime I will start working on optimization. I expect that it will > require API changes in IPageStore interface so it may be only for Wicket 7.
