ericflo / django-tokyo-sessions Goto Github PK

The session backend currently dies with an exception if a session that is being deleted doesn't exist in Tokyo Tyrant any more for some reason.

Here's a patch to ignore the error:

diff --git a/tokyo_sessions/tyrant.py b/tokyo_sessions/tyrant.py
index 2152630..22cb54b 100644
--- a/tokyo_sessions/tyrant.py
+++ b/tokyo_sessions/tyrant.py
@@ -62,4 +62,8 @@ class SessionStore(SessionBase):
             if self._session_key is None:
                 return
             session_key = self._session_key
-        del get_server()[session_key]
\ No newline at end of file
+        try:
+            del get_server()[session_key]
+        except:
+            # Ignore error if session didn't exist in database any more.
+            pass

(copied over from the PyTyrant ticket tracker)

We're testing a Django app with django-tokyo-sessions. At a certain point
in our stress tests pytyrant starts throwing a 99 error. Sounds like low
level UNIX. We increased the amount of threads to 24 (on a beefy machine).
50 concurrent requests in our test suite. We also tried to use memory
storage. Started throwing the error around the 9000 request count mark at
around 200 completed requests per second.

Traceback:

http://dpaste.com/hold/46340/