Moved some WANObjectCache number literals to constants
authorAaron Schulz <aschulz@wikimedia.org>
Wed, 16 Sep 2015 17:05:54 +0000 (10:05 -0700)
committerAaron Schulz <aschulz@wikimedia.org>
Wed, 16 Sep 2015 17:05:54 +0000 (10:05 -0700)
Change-Id: I848440408b54c1dfd4961c15cc83aad9a1a19425

includes/libs/objectcache/WANObjectCache.php

index dcd413d..56c544f 100644 (file)
@@ -73,6 +73,10 @@ class WANObjectCache {
        const CHECK_KEY_TTL = 31536000; // 1 year
        /** Seconds to keep lock keys around */
        const LOCK_TTL = 5;
+       /** Default remaining TTL at which to consider pre-emptive regeneration */
+       const LOW_TTL = 10;
+       /** Default TTL for temporarily caching tombstoned keys */
+       const TEMP_TTL = 5;
 
        /** Idiom for set()/getWithSetCallback() TTL */
        const TTL_NONE = 0;
@@ -473,9 +477,9 @@ class WANObjectCache {
        final public function getWithSetCallback(
                $key, $callback, $ttl, array $checkKeys = array(), array $opts = array()
        ) {
-               $lowTTL = isset( $opts['lowTTL'] ) ? $opts['lowTTL'] : min( 10, $ttl );
+               $lowTTL = isset( $opts['lowTTL'] ) ? $opts['lowTTL'] : min( self::LOW_TTL, $ttl );
                $lockTSE = isset( $opts['lockTSE'] ) ? $opts['lockTSE'] : -1;
-               $tempTTL = isset( $opts['tempTTL'] ) ? $opts['tempTTL'] : 5;
+               $tempTTL = isset( $opts['tempTTL'] ) ? $opts['tempTTL'] : self::TEMP_TTL;
 
                // Get the current key value
                $curTTL = null;