[livejournal] r16527: Added hack to load subscripitions from D...

[vadvs]

Committer: vsukhanovAdded hack to load subscripitions from DB slave

U trunk/cgi-bin/LJ/Event.pm

Comments

I'll admit my LJ codebase knowledge is 5+ years out of date

[halkeye]

Would the require statement not cause a new module to be loaded, and then unshare memory between all the apache nodes? AFAIK Anything loaded / changed after apache forks is not shared between processes and increases memory footprint.

Re: I'll admit my LJ codebase knowledge is 5+ years out of date

[vadvs]

Should we share this module between all apache processes unconditionally? This code is called rather rare and probability that it would be loaded twice on the same server simultaneously very small.