Use a real PHP array for the cache#29
Use a real PHP array for the cache#29joshuaadickerson wants to merge 3 commits intoAOEpeople:masterfrom
Conversation
This creates an array instead of using a serialized object for the cache. This should be more efficient.
I didn't realize I had to support PHP 5.3
|
Hi It would be quite interesting to see the performance test on the current PHP 7, to see if this change gives any performance improvement at all. |
|
Benchmarks know best, but that is illogical to me. PHP has to unserialize the string before it can use it. Whereas with a PHP file, APC will cache it as a file and PHP will use it as a file. The individual process doesn't have to do any work. It will probably take longer to create the string, but read performance is much more important than write. |
|
@joshuaadickerson I'm sure you agree that it doesn't make sense to argue about the feeling what "should" be faster and why. Please provide benchmark proving your solution is faster then the current one. |
This creates an array instead of using a serialized object for the cache. This should be more efficient.