https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16579 --- Comment #46 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- (In reply to Jacek Ablewicz from comment #42)
(In reply to Kyle M Hall from comment #41)
(In reply to Nick Clemens from comment #40)
I haven't fully tested, but initial read through and applying looks good. This makes it much easier to get memcached functioning with plack and is a good improvement
I agree! This is a huge improvement and greatly simplifies the setup.
Jacek, what do you think?
I think generally it's good, and certainly good enough for the 'promotion' from "In discussion" to "Needs sign-off". I have some doubts re/ certain parts (mostly about an introduction of that 'namespace' thingy to the Koha caching system - not entirely sure if it's strictly necessary, and - in case it is - if doing it in this particular way is the best way possible).
I agree, that's why I have added to FIXME in the commit message since the first submission :) FIXME: There is an architecture problem here: the L1 cache should be defined in Koha::Cache All the patches have been splited into 3 bug reports. Letting this one for the general discussion about these patches. -- You are receiving this mail because: You are watching all bug changes.