brianchandotcom/liferay-portal — pull request #6578 by shuyangzhou (2012-08-22 00:39:17+00:00)
Title: LPS-29266 Fix performance regression

Only two files change, the PACLAdvice one is undoing previous change, so ignore it.

The service_base_impl.ftl one is switching context classloader on receiver side, by getting service's own define classloader(You should always have permission to do so).

I am not using the PACLClassLoaderUtil, which means no SecurityManager logic will be shortcut. This may cause it a little bit slower for PACL + CLP, but both PACL and CLP are very slow already, it won't make a difference. But it can improve the general performance when not using CLP, even when PACL is on, and when PACL is off, there is no additional overhead at all.

So basically this is making a rarely used slow path more slower, but making most often used paths much more faster.