[Catalyst-commits] r10542 - in trunk/Catalyst-Plugin-Session-Store-FastMmap: . lib/Catalyst/Plugin/Session/Store t/lib

[kmx at dev.catalyst.perl.org]

Author: kmx
Date: 2009-06-15 19:04:33 +0000 (Mon, 15 Jun 2009)
New Revision: 10542

Modified:
   trunk/Catalyst-Plugin-Session-Store-FastMmap/Changes
   trunk/Catalyst-Plugin-Session-Store-FastMmap/lib/Catalyst/Plugin/Session/Store/FastMmap.pm
   trunk/Catalyst-Plugin-Session-Store-FastMmap/t/lib/SessionStoreTest.pm
Log:
C::P::Session::Store::FastMmap - rev.10535 back again + little doc patch

Modified: trunk/Catalyst-Plugin-Session-Store-FastMmap/Changes
===================================================================
--- trunk/Catalyst-Plugin-Session-Store-FastMmap/Changes	2009-06-15 16:19:39 UTC (rev 10541)
+++ trunk/Catalyst-Plugin-Session-Store-FastMmap/Changes	2009-06-15 19:04:33 UTC (rev 10542)
@@ -1,4 +1,5 @@
-        - Create file paths correctly in Win32
+        - Fixed warning during basic.t on Win32
+        - Added comment into doc about not being "thread-safe" 
 
 0.10   13 May 2009
         - Change tests to be compatible with Catalyst 5.80004

Modified: trunk/Catalyst-Plugin-Session-Store-FastMmap/lib/Catalyst/Plugin/Session/Store/FastMmap.pm
===================================================================
--- trunk/Catalyst-Plugin-Session-Store-FastMmap/lib/Catalyst/Plugin/Session/Store/FastMmap.pm	2009-06-15 16:19:39 UTC (rev 10541)
+++ trunk/Catalyst-Plugin-Session-Store-FastMmap/lib/Catalyst/Plugin/Session/Store/FastMmap.pm	2009-06-15 19:04:33 UTC (rev 10542)
@@ -140,11 +140,17 @@
 expired prematurely, due to the LRU caching policy employed by
 L<Cache::FastMmap>. To get around this you can increase the C<cache_size>
 parameter, or switch session storage backends.
+L<Cache::FastMmap> defaults to around 5mb (89 * 64k).
 
 This is particularly inappropriate for use as a backend for e.g.
 L<Catalyst::Plugin::Session::PerUser>, for example.
 
-L<Cache::FastMmap> defaults to around 5mb (89 * 64k).
+As L<Cache::FastMmap> is not "thread-safe" (at least version 1.30 and before)
+therefore also this module does not work in multi-threaded enviroment. 
+It is "fork-safe", however keep in mind that on Win32 the perl "fork" call is 
+implemented as an emulation via threads - that is the reason why you cannot use
+this store for example when running you catalyst application on Win32 platform 
+with L<Catalyst::Engine::HTTP::Prefork> engine. 
 
 =head1 CONFIGURATION
 

Modified: trunk/Catalyst-Plugin-Session-Store-FastMmap/t/lib/SessionStoreTest.pm
===================================================================
--- trunk/Catalyst-Plugin-Session-Store-FastMmap/t/lib/SessionStoreTest.pm	2009-06-15 16:19:39 UTC (rev 10541)
+++ trunk/Catalyst-Plugin-Session-Store-FastMmap/t/lib/SessionStoreTest.pm	2009-06-15 19:04:33 UTC (rev 10542)
@@ -8,7 +8,9 @@
 
 use Catalyst::Plugin::Session::Test::Store (
     backend => "FastMmap",
-    config  => { storage => File::Spec->catfile(File::Temp::tempdir( 'sessionstoretestXXXX', CLEANUP => 1 ),  'storage_file') },
+    config => { storage => scalar(File::Temp::tmpnam()) }, 
+    # we do not care in this package about deleting temporary file as it is
+    # removed automatically by Cache::FastMmap 
     extra_tests => 1
 );