[MDL-57570] Add the option to switch the serializer to igbinary in static cache store - Moodle Tracker

XML

Word

Printable

Details

Type: Improvement
Status: Closed
Priority: Minor
Resolution: Fixed
Affects Version/s: 3.2, 3.3
Fix Version/s: 3.3
Component/s: Caching, Performance
Labels:
- ci
- release_notes

Affected Branches:
MOODLE_32_STABLE, MOODLE_33_STABLE
Fixed Branches:
MOODLE_33_STABLE
Pull from Repository:
https://github.com/mwehr/moodle
Pull Main Branch:
~~MDL-57570~~-master
Pull Main Diff URL:
https://github.com/mwehr/moodle/commit/7ca1161645c618b49b5593a6cbbd4f7d028ff790
Testing Instructions:

Hide

Install the PHP igbinary module.
Enable the igbinary extension in the php igbinary extension config.
The static cache should use the igbinary_un/serialize functions.
Executing vendor/bin/phpunit cachestore_static_test cache/stores/static/tests/static_test.php should lead to 74 assertions passed.

Disable the igbinary extension in the php igbinary extension config.
The static cache should use the PHP un/serialize functions.
Executing vendor/bin/phpunit cachestore_static_test cache/stores/static/tests/static_test.php should lead to 1 skipped unit test.

Show
Install the PHP igbinary module. Enable the igbinary extension in the php igbinary extension config. The static cache should use the igbinary_un/serialize functions. Executing vendor/bin/phpunit cachestore_static_test cache/stores/static/tests/static_test.php should lead to 74 assertions passed. Disable the igbinary extension in the php igbinary extension config. The static cache should use the PHP un/serialize functions. Executing vendor/bin/phpunit cachestore_static_test cache/stores/static/tests/static_test.php should lead to 1 skipped unit test.

Description

Hi,

I think the option to use the igbinary serializer in the static cache store would be a handy feature to reduce the memory footprint when processing requests .
According to some quick tests the average cache size could be reduced by 40% - 50%.

Attachments

Issue Links

has been marked as being related by

MDL-57572 Add the option to switch the serializer to igbinary in Redis cache store

Closed

Activity

People

Assignee:: Mario Wehr

Reporter:: Mario Wehr

Peer reviewer:: Matteo Scaramuccia

Integrator:: Eloy Lafuente (stronk7)

Tester:: Rajesh Taneja

Participants:: Andrew Lyons, CiBoT, Dan Marsden, Eloy Lafuente (stronk7), Mario Wehr, Matteo Scaramuccia, Rajesh Taneja

Votes:: 0 Vote for this issue

Watchers:: 5 Start watching this issue

Dates

Created:: 05/Jan/17 8:22 AM

Updated:: 27/Aug/23 8:33 PM

Resolved:: 19/Jan/17 11:52 PM

Fix Release Date:: 15/May/17