search

AArhin / memcached-session-manager

Automatically exported from code.google.com/p/memcached-session-manager
0 stars 0 forks source link

Check if session backup is necessary triggers a memcached lookup #2

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
Right now the SessionTrackerValve checks if session backup is necessary by
asking for the current session at the end of the request. If there's a
sessoin existing, the session backup must be performed.

Unfortunately, this leads to MemcachedBackupSessionManager.findSession
which performs a loadFromMemcached. This is not ideal in terms of
performance and should be changed.
E.g. the request and response cookies could be used for this.

Original issue reported on code.google.com by martin.grotzke on 10 Apr 2009 at 9:39

GoogleCodeExporter commented 9 years ago 
Optimization implemented, checking the requestedSessionId and the response 
cookie.

Original comment by martin.grotzke on 25 Apr 2009 at 10:25

GoogleCodeExporter commented 9 years ago 
Therefore fixed.

Original comment by martin.grotzke on 25 Apr 2009 at 10:26