Cache db queries to appconfig table

[karlitschek]

We do over 50 selects of the appconfig table. This is way to much can should be cached. A good option would be to store the already read data in an array in memory. The data is read from the database completely at first request.

• Load the app config table and store it in the memory.
• Change the config class to work on the memory array
• Write requests should be written to the db and the cache should be flushed.

[PVince81]

Sounds like a good plan :-)

[karlitschek]

@icewind1991 Can you implement this before the feature freeze? Would be great.

[DeepDiver1975]

AFAIK this is already done
https://github.com/owncloud/core/blob/master/lib/private/appconfig.php#L50

[karlitschek]

@DeepDiver1975 I saw the code too. But it doesn´t seems to work. I can see a lot of queries to the appconfig table. But there should be only one.

[DeepDiver1975]

These function do not use the cache:
https://github.com/owncloud/core/blob/master/lib/private/appconfig.php#L226
https://github.com/owncloud/core/blob/master/lib/private/appconfig.php#L98

[PVince81]

@icewind1991 can this be closed or is there anything else to be done ?

[icewind1991]

I think this can be closed