You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by "Stefan Fußenegger (JIRA)" <ji...@apache.org> on 2008/06/02 12:03:45 UTC
[jira] Commented: (WICKET-1679) Memory leak in DiskPageStore
[ https://issues.apache.org/jira/browse/WICKET-1679?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12601576#action_12601576 ]
Stefan Fußenegger commented on WICKET-1679:
-------------------------------------------
see also http://www.nabble.com/Memory-leak-in-DiskPageStore-td17597466.html
> Memory leak in DiskPageStore
> ----------------------------
>
> Key: WICKET-1679
> URL: https://issues.apache.org/jira/browse/WICKET-1679
> Project: Wicket
> Issue Type: Bug
> Components: wicket
> Affects Versions: 1.3.3
> Reporter: Stefan Fußenegger
> Original Estimate: 0.25h
> Remaining Estimate: 0.25h
>
> No entries are removed from DiskPageStore.sessionIdToEntryMap. I guess, this should be added to the DiskPageStore.unbind(String sessionId) method:
> public void unbind(String sessionId)
> {
> // FIX: replace get() with remove()
> SessionEntry entry = (SessionEntry)sessionIdToEntryMap.get(sessionId);
> if (entry != null)
> {
> if (isSynchronous())
> {
> entry.unbind();
> }
> else
> {
> List pages = getPagesToSaveList(sessionId);
> synchronized (pages)
> {
> flushPagesToSaveList(sessionId, pages);
> entry.unbind();
> }
> pagesToSaveAll.remove(sessionId);
> }
> }
> }
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.