Comment 6 for bug 1449709

Hello, I tested this out on master as of 6/9/2015.

It seems to just work, I did several measurements with curl before and after and there is a definite improvement in load times with these changes. It would be better to do some longer term testing and graph it all out, if I find time I'll plug it into zabbix to graph a days worth of latency before and after the patch.

Without caching
real 0m2.021s
real 0m2.152s
real 0m2.320s
real 0m1.586s
real 0m2.117s

With caching
real 0m1.738s
real 0m1.935s
real 0m2.000s
real 0m2.065s
real 0m1.829s
real 0m0.964s
real 0m1.724s
real 0m0.916s
real 0m0.928s
real 0m1.663s
real 0m1.172s
real 0m0.969s
real 0m0.914s

The only oddity I noticed was that I couldn't get LP#1435938 "Allow Staff to clear Added Content cache" to work, but I really don't know if that is related to my test system or to some interaction with this. It worked fine on the sandbox setup to test it. And it still doesn't work when I disable the caching by commenting out "PerlSetVar OILSWebCompiledTemplateCache" in eg_vhost.conf. So it probably isn't related.

I have tested this code and consent to signing off on it with my name, Josh Stompro and email address, <email address hidden>.