You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by "Sven Meier (JIRA)" <ji...@apache.org> on 2018/07/07 20:02:00 UTC

[jira] [Comment Edited] (WICKET-6563) Rework page and data storage

    [ https://issues.apache.org/jira/browse/WICKET-6563?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16530471#comment-16530471 ] 

Sven Meier edited comment on WICKET-6563 at 7/7/18 8:01 PM:
------------------------------------------------------------

I'm working on a new implementation that is based on a chain of IPageStores. The current implementation will be reorganized as follows:

 
||Wicke 8.x||Wicket 9.x||notes||
|IPageManager|IPageManager| |
|PageStoreManager|PageManager|very simple facade|
| |RequestPageStore|request cache as first IPageStore in the chain|
| | | |
|IPageManagerContext|IPageContext|now passed to IPageStore methods instead of constructor|
|DefaultPageManagerContext|DefaultPageContext| |
| | | |
|IPageStore|IPageStore| |
|AsynchronousPageStore|AsynchronousPageStore| |
|DefaultPageStore|InSessionPageStore|stores pages in session|
|PerSessionPageStore|-| |
|InMemoryPageStore|InMemoryPageStore|stores pages in memory|
|-|GroupingPageStore|allows grouping of pages|
| | | |
|IDataStore|-|no longer present|
|AsynchronousDataStore|-|no longer present|
|DiskDataStore|DiskPageStore|receives pages now|
|HttpSessionDataStore|-|subsumed by InMemoryPageStore|
| | | |
|-|SerializedPage|holds a serialized page, can be passed between stores|
|PageWindowDescription|IPersistedPage|information about persisted pages in DiskPageStore and InMemoryPageStore too|

 


was (Author: svenmeier):
I'm working on a new implementation that is based on a chain of IPageStores. The current implementation is reorganized as follows:

 
||Wicke 8.x||Wicket 9.x||notes||
|IPageManager|IPageManager| |
|PageStoreManager|PageManager|very simple facade|
| |RequestPageStore|request cache as first IPageStore in the chain|
| | | |
|IPageManagerContext|IPageContext|now passed to IPageStore methods instead of constructor|
|DefaultPageManagerContext|DefaultPageContext| |
| | | |
|IPageStore|IPageStore| |
|AsynchronousPageStore|AsynchronousPageStore| |
|DefaultPageStore|InSessionPageStore|stores pages in session|
|PerSessionPageStore|-| |
|InMemoryPageStore|InMemoryPageStore|stores pages in memory|
|-|GroupingPageStore|allows grouping of pages|
| | | |
|IDataStore|-|no longer present|
|AsynchronousDataStore|-|no longer present|
|DiskDataStore|DiskPageStore|receives pages now|
|HttpSessionDataStore|-|subsumed by InMemoryPageStore|
| | | |

 

> Rework page and data storage
> ----------------------------
>
>                 Key: WICKET-6563
>                 URL: https://issues.apache.org/jira/browse/WICKET-6563
>             Project: Wicket
>          Issue Type: Improvement
>          Components: wicket
>    Affects Versions: 9.0.0
>            Reporter: Sven Meier
>            Assignee: Sven Meier
>            Priority: Minor
>
> IPageManager, IPageStore and IDataStore are riddled with complicated and error-prone code aiming to do one simple thing: keeping pages around.
> We should rework that.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)