Skip to content
This repository has been archived by the owner on Jan 13, 2022. It is now read-only.

Add Memcached support for the MediaWiki target #87

Merged
merged 1 commit into from
Jan 13, 2020

Conversation

octmoraru
Copy link
Contributor

This diff adds support for Memcached. If a target supports Memcached,
oss-performance will start it on 127.0.0.1:11888. The only target supporting
Memcached right now is MediaWiki. For other targets to support Memcached, they
must override PerfTarget::supportsMemcached().
This diff also sets $wgUseDatabaseMessages to false in the configuration file
if Memcached is disabled, as recommended in the MediaWiki manual.

For MediaWiki with Memcached enabled, observing a ~6% increase in RPS, a
decrease in system idle time, as well as ~7 percentage points more cycles being
spent in HHVM.

Command line options added:

--memcached: path to memcached binary
--delay-memcached-startup: number of seconds to delay Memcached startup
--memcached-port: TCP port to listen on (default is 11888)
--no-memcached: don't use memcached (even if target supports it)
--memcached-threads: number of memcached threads to spawn (default is 4 for
machines with small number of cores, 32 otherwise)

This diff adds support for Memcached. If a target supports Memcached,
oss-performance will start it on 127.0.0.1:11888. The only target supporting
Memcached right now is MediaWiki. For other targets to support Memcached, they
must override PerfTarget::supportsMemcached().
This diff also sets $wgUseDatabaseMessages to false in the configuration file
if Memcached is disabled, as recommended in the MediaWiki manual.

For MediaWiki with Memcached enabled, observing a ~6% increase in RPS, a
decrease in system idle time, as well as ~7 percentage points more cycles being
spent in HHVM.

Command line options added:

--memcached: path to memcached binary
--delay-memcached-startup: number of seconds to delay Memcached startup
--memcached-port: TCP port to listen on (default is 11888)
--no-memcached: don't use memcached (even if target supports it)
--memcached-threads: number of memcached threads to spawn (default is 4 for
machines with small number of cores, 32 otherwise)
@meteorfox meteorfox changed the base branch from master to staging January 13, 2020 23:50
@meteorfox meteorfox merged commit 917a47f into facebookarchive:staging Jan 13, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants