Make JobQueueRedis stat calls match the DB one
authorAaron Schulz <aschulz@wikimedia.org>
Tue, 20 Oct 2015 21:24:09 +0000 (14:24 -0700)
committerAaron Schulz <aschulz@wikimedia.org>
Tue, 20 Oct 2015 21:24:09 +0000 (14:24 -0700)
It looks like someone just forgot to update these to
the newer style (e.g. sans "job-").

Change-Id: Idf953bb73ad31425c5c366d4b2169a42bd5e022d

includes/jobqueue/JobQueueRedis.php

index 29c8068..002fe0d 100644 (file)
@@ -301,7 +301,7 @@ LUA;
                                        break; // no jobs; nothing to do
                                }
 
-                               JobQueue::incrStats( 'job-pop', $this->type );
+                               JobQueue::incrStats( 'pops', $this->type );
                                $item = $this->unserialize( $blob );
                                if ( $item === false ) {
                                        wfDebugLog( 'JobQueueRedis', "Could not unserialize {$this->type} job." );
@@ -393,7 +393,7 @@ LUA;
                                return false;
                        }
 
-                       JobQueue::incrStats( 'job-ack', $this->type );
+                       JobQueue::incrStats( 'acks', $this->type );
                } catch ( RedisException $e ) {
                        $this->throwRedisException( $conn, $e );
                }