Export to GitHub

memcached-session-manager - issue #229

I do not think MemcachedBackupSessionManager.generateSessionId() need to be synchronized.

Posted on Jul 21, 2015 by Happy Bear

I have reviewed the code, and as title. Do you agree?

Comment #1

Posted on Jul 21, 2015 by Grumpy Bear

Yes, AFAICS you're right. Probably at the time I wrote this some tomcat session manager I used for inspiration had this synchronized so that I just "copied" it.

Comment #2

Posted on Jul 21, 2015 by Happy Bear

So, it could be faster a bit now.:)

Comment #3

Posted on Jul 21, 2015 by Grumpy Bear

Thanks, I pushed the change to master.

Status: Fixed

Labels:
Type-Enhancement Priority-Medium 1.8.4