[Koha-devel] Why is config ($KOHA_CONF) stored in memcached ?
Julian Maurice
julian.maurice at biblibre.com
Sun May 2 14:52:24 CEST 2021
Patch is ready to be tested at
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28276 ;)
Le 01/05/2021 à 14:53, Tomas Cohen Arazi a écrit :
> I vote for fixing it so the file is only hit the first time.
>
> El vie, 30 abr 2021 a las 14:19, Julian Maurice
> (<julian.maurice at biblibre.com <mailto:julian.maurice at biblibre.com>>)
> escribió:
>
> Hi all,
>
> Maybe it's a dumb question, but I don't understand why the config is
> stored in memcached as:
> - memcached server address is in this config, so we need to read
> $KOHA_CONF file first in order to connect to memcached server
> - config is kept in memory forever ($C4::Context::context), so we
> shouldn't need the cache once starman has started.
>
> If you add a warning in Koha::Config::read_from_file, you'll see that
> the file is read 3 times per worker, even when the config already exist
> in cache (it's read 4 times when the cache is empty).
> It's read each time we call Koha::Cache->new (+ in C4::Context->new if
> the cache is empty)
>
> Shouldn't we read the file only in C4::Context->new (without using
> cache), and make Koha::Cache use C4::Context->config instead ?
>
> --
> Julian Maurice
> BibLibre
> _______________________________________________
> Koha-devel mailing list
> Koha-devel at lists.koha-community.org
> <mailto:Koha-devel at lists.koha-community.org>
> https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-devel
> <https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-devel>
> website : https://www.koha-community.org/
> <https://www.koha-community.org/>
> git : https://git.koha-community.org/ <https://git.koha-community.org/>
> bugs : https://bugs.koha-community.org/
> <https://bugs.koha-community.org/>
>
>
>
> --
> Tomás Cohen Arazi
> Theke Solutions (http://theke.io <http://theke.io/>)
> ✆ +54 9351 3513384
> GPG: B2F3C15F
--
Julian Maurice
BibLibre
More information about the Koha-devel
mailing list