jenkins-bot has submitted this change and it was merged.
Change subject: Made User actually use the mQuickTouched process cache
......................................................................
Made User actually use the mQuickTouched process cache
Change-Id: I158eae2dac16b5fdacd095fff7fb031b42804a1e
---
M includes/User.php
1 file changed, 3 insertions(+), 1 deletion(-)
Approvals:
Legoktm: Looks good to me, approved
jenkins-bot: Verified
diff --git a/includes/User.php b/includes/User.php
index 2f9b716..78693c1 100644
--- a/includes/User.php
+++ b/includes/User.php
@@ -2309,7 +2309,9 @@
if ( $this->mQuickTouched === null ) {
$key = wfMemcKey( 'user-quicktouched', 'id',
$this->mId );
$timestamp = $wgMemc->get( $key );
- if ( !$timestamp ) {
+ if ( $timestamp ) {
+ $this->mQuickTouched = $timestamp;
+ } else {
# Set the timestamp to get HTTP 304
cache hits
$this->touch();
}
--
To view, visit https://gerrit.wikimedia.org/r/200115
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I158eae2dac16b5fdacd095fff7fb031b42804a1e
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: Aaron Schulz <[email protected]>
Gerrit-Reviewer: Legoktm <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
