search

picketlink2 / federation

PicketLink v2.1.x Federation. PicketLink v2.5.x onwards is hosted at https://github.com/picketlink
http://www.picketlink.org
14 stars 29 forks source link

Picketlink performance is too bad due to have unresonable lock. #165

Open wzhx78 opened 11 years ago

wzhx78 commented 11 years ago

when we test picketlink performance, we found the picketlink performance is very bad due to each of process have global lock in the code. e.g org.picketlink.identity.federation.web.process.SAMLHandlerChainProcessor. ServiceProviderBaseProcessor and org/picketlink/identity/federation/bindings/tomcat/idp/AbstractIDPValve. but we check the handler is thread safe, so the lock is unreasonable. we can remove this lock or change the hanlder not singletone.

we had tried to fix by our self. after remove unreasonable lock, the TPS will back to 456, one TPS include one login and one global logout.

please check our gist.

https://gist.github.com/wzhx78/5418959

anilsaldhana commented 11 years ago

https://issues.jboss.org/browse/PLINK-249

pedroigor commented 11 years ago

Updated https://issues.jboss.org/browse/PLINK-249