[dev] Re: [cvs] commit: horde/config conf.xml framework/Horde/Horde
Registry.php
Michael M Slusarz
slusarz at mail.curecanti.org
Thu Mar 24 14:18:04 PST 2005
Quoting Jan Schneider <jan at horde.org>:
> Zitat von Jan Schneider <jan at horde.org>:
>
>> Zitat von Michael M Slusarz <slusarz at bigworm.curecanti.org>:
>>
>>> Quoting Jan Schneider <jan at horde.org>:
>>>
>>>> Zitat von Michael M Slusarz <slusarz at curecanti.org>:
>>>>
>>>>> slusarz 2005-03-23 16:32:20 PST
>>>>>
>>>>> Modified files:
>>>>> config conf.xml
>>>>> Horde/Horde Registry.php
>>>>> Log:
>>>>> Implement session caching of some registry information. See conf.xml ->
>>>>> 'registry_cache' entry for description of what is cached.
>>>>
>>>> Cool, that should give a huge perfomance boost. Unfortunately also a
>>>> memory boost, but that's another story. Did you do any benchmarks?
>>>
>>> Not yet. And in my haste I forgot to add to the conf.xml description
>>> that "Additionally, caching of registry information will result in
>>> slightly larger session sizes". But disk space is cheap, I/O access
>>> times generally are not, so I expect there to be a noticeable
>>> difference (1-2%??), especially for something like the sidebar which
>>> has to parse multiple configuration files.
>>
>> Forget it. I actually was talking about memory, not disk space. But the
>> data gets loaded into memory anyway, whether you use caching or not, so
>> that doesn't make a difference. The disk space is negligible.
>>
>> I actually expect an even higher performance gain, because when I did
>> some profiling recently, the registry instantiation was one of the hogs
>> on light or highly optimized pages.
>
> Ah, something I forgot to ask. Shouldn't it be sufficient to check the
> registry.php mtime once per request to eliminate the disadvantage of
> not being able to change the registry at runtime?
This could be the next step. Although I thought doing time requests on
files is a horrendously slow operation that would make negligible gains
achieved by skipping loading the file. Then again, it probably is
worth it. Having to do it on config files, though, starts adding back
to the overhead quite a bit...
michael
_______________________________________
Michael Slusarz [slusarz at curecanti.org]
More information about the dev
mailing list