[Koha-patches] [PATCH] cache the session for the duration of the script
Michael Hafen
mdhafen at tech.washk12.org
Mon May 10 20:18:06 CEST 2010
Spoke too soon. The patch broke Staging Files for Import. I'll have to
do something else I guess.
On Mon, 2010-05-10 at 09:53 -0400, Galen Charlton wrote:
> Hi,
>
> On Mon, May 10, 2010 at 9:47 AM, Michael Hafen <mdhafen at tech.washk12.org> wrote:
> > I don't have any profile data, it's just a supposition. I figure it
> > will save however many milliseconds it takes to parse the session data
> > again to form the object.
>
> My concern with that is that a micro-improvement to performance can
> cause an unexpected breakage in semantics by adding an extra layer of
> caching. Some of this is a theoretical concern - for example, in a
> mod_perl setup, a package-level cache of the session would break
> per-user session management badly. Of course, since we don't run Koha
> using full mod_perl, that isn't a direct issue. But this close to
> release, I'm reluctant to add package-level caching of anything absent
> a significant performance boost.
>
> Of course, if you track down your premature session timeouts issue,
> that could be a different story.
>
> Regards,
>
> Galen
--
Michael Hafen
Systems Analyst and Programmer
Washington County School District
Utah, USA
for Koha checkout
http://development.washk12.org/gitweb/
or
git://development.washk12.org/koha
More information about the Koha-patches
mailing list