I do not think MemcachedBackupSessionManager.generateSessionId() need to be synchronized.
GoogleCodeExporter opened this issue · comments
I have reviewed the code, and as title. Do you agree?
Original issue reported on code.google.com by qian...@gmail.com
on 21 Jul 2015 at 1:41
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.
Original comment by martin.grotzke
on 21 Jul 2015 at 8:07
So, it could be faster a bit now.:)
Original comment by qian...@gmail.com
on 21 Jul 2015 at 10:37
Thanks, I pushed the change to master.
Original comment by martin.grotzke
on 21 Jul 2015 at 9:43
- Changed state: Fixed
- Added labels: 1.8.4, Type-Enhancement
- Removed labels: Type-Defect