From 58c8e2b969457b6945533d966b265a3804f11f36 Mon Sep 17 00:00:00 2001 From: Aaron Schulz Date: Thu, 12 Mar 2015 18:06:28 -0700 Subject: [PATCH] Made getNewtalk use slaves instead of cache/master * This was causing floods of master connections on page views Bug: T92357 Change-Id: I02fd1505f589112b8f6217047e6713845142a1c3 --- includes/User.php | 12 +----------- 1 file changed, 1 insertion(+), 11 deletions(-) diff --git a/includes/User.php b/includes/User.php index a4f6b77bab..c811dd5d29 100644 --- a/includes/User.php +++ b/includes/User.php @@ -2011,17 +2011,7 @@ class User implements IDBAccessObject { // Anon newtalk disabled by configuration. $this->mNewtalk = false; } else { - global $wgMemc; - $key = wfMemcKey( 'newtalk', 'ip', $this->getName() ); - $newtalk = $wgMemc->get( $key ); - if ( strval( $newtalk ) !== '' ) { - $this->mNewtalk = (bool)$newtalk; - } else { - // Since we are caching this, make sure it is up to date by getting it - // from the master - $this->mNewtalk = $this->checkNewtalk( 'user_ip', $this->getName(), true ); - $wgMemc->set( $key, (int)$this->mNewtalk, 1800 ); - } + $this->mNewtalk = $this->checkNewtalk( 'user_ip', $this->getName() ); } } else { $this->mNewtalk = $this->checkNewtalk( 'user_id', $this->mId ); -- 2.20.1