[jira] Issue Comment Edited: (WICKET-1837) DiskPageStore: 32k directory entries.

["Bruno Borges (JIRA)" <ji...@apache.org>]

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

miojo edited comment on WICKET-1837 at 11/13/08 5:37 AM:
----------------------------------------------------------------

I would add this change to this patch:

@@ -385,6 +408,8 @@

- 		sessionId = sessionId.replace('*', '_');
- 		sessionId = sessionId.replace('/', '_');
+ 		sessionId = Files.cleanupFilename(String)

Regards,
Bruno

      was (Author: miojo):
    I would add this change to this patch:

sessionId = Files.cleanupFilename(sessionId);

@see org.apache.wicket.util.file.Files#cleanupFilename(String)

Regards,
Bruno
  
> DiskPageStore: 32k directory entries.
> -------------------------------------
>
>                 Key: WICKET-1837
>                 URL: https://issues.apache.org/jira/browse/WICKET-1837
>             Project: Wicket
>          Issue Type: Improvement
>          Components: wicket
>    Affects Versions: 1.4-M3
>         Environment: Caucho Resin 3.2
>            Reporter: uwe schaefer
>            Assignee: Matej Knopp
>             Fix For: 1.4-RC2
>
>         Attachments: sessionId_deep_path_revisited.patch
>
>
> When using JMeter to battletest a Wicket-App, i saw 31999 directories created (one per session) in the tmp dir to store the pagemaps.
> This is a problem because the underlying filesystem might (and does in my case) prevent wicket from creating more directory entries than that.
> i appended a very very simple patch to address this by creating 
> <tmp>/ab/cd/ef instead of <tmp>/abcdef for pagemap storage.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.