Aaron Schulz has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/394772 )
Change subject: Only send statsd data for WAN cache in non-CLI mode
......................................................................
Only send statsd data for WAN cache in non-CLI mode
Bug: T181385
Change-Id: Ic51a084206cea93621393359ded4ff77beef5f13
---
M includes/objectcache/ObjectCache.php
1 file changed, 3 insertions(+), 2 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/core
refs/changes/72/394772/1
diff --git a/includes/objectcache/ObjectCache.php
b/includes/objectcache/ObjectCache.php
index 07432c0..67d2346 100644
--- a/includes/objectcache/ObjectCache.php
+++ b/includes/objectcache/ObjectCache.php
@@ -342,14 +342,15 @@
$params['channels'][$action] = $channel;
}
$params['cache'] = self::newFromParams( $params['store'] );
- $params['stats'] = $services->getStatsdDataFactory();
if ( isset( $params['loggroup'] ) ) {
$params['logger'] = LoggerFactory::getInstance(
$params['loggroup'] );
} else {
$params['logger'] = LoggerFactory::getInstance(
'objectcache' );
}
- // Let pre-emptive refreshes happen post-send on HTTP requests
if ( !$wgCommandLineMode ) {
+ // Send the statsd data post-send on HTTP requests;
avoid in CLI mode (T181385)
+ $params['stats'] = $services->getStatsdDataFactory();
+ // Let pre-emptive refreshes happen post-send on HTTP
requests
$params['asyncHandler'] = [ DeferredUpdates::class,
'addCallableUpdate' ];
}
$class = $params['class'];
--
To view, visit https://gerrit.wikimedia.org/r/394772
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic51a084206cea93621393359ded4ff77beef5f13
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: Aaron Schulz <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
