pmouawad opened a new pull request #376:
URL: https://github.com/apache/tomcat/pull/376


   Hello,
   First thanks for your great product I've been using for years.
   
   Recently, I worked on a project that uses JDBCStore to store sessions.
   I had to run performance tests on it.
   Those tests revealed a high contention point in this component due to the 
coarse granularity of synchronization. 
   The impact was that 1 thread could block hundreds of others leading to high 
response time at higher scale.
   
   So I implemented (based on existing one) a new one with following 
differences:
   
   - I kept only the dataSourceName based configuration
   - I removed stateful instance variables in order to be able to lift 
synchronization
   
   This new implementation improved drastically performances.
   
   I am aware that there is duplication with JDBCStore that could be improved 
but it's a first shot and I lack of time unfortunately.
   
   I hope you can take into account this contribution.
   
   The associated bugzilla enhancement is:
   
   - https://bz.apache.org/bugzilla/show_bug.cgi?id=64877
   
   Regards
   Philippe M.
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to