Addshore has uploaded a new change for review.
https://gerrit.wikimedia.org/r/300913
Change subject: Only send global betafeature metric if whitelisted
......................................................................
Only send global betafeature metric if whitelisted
Change-Id: I249f247a7d11b4bd173a08f3e00ff751855d65a6
---
M src/betafeatures/counts.php
1 file changed, 6 insertions(+), 1 deletion(-)
git pull ssh://gerrit.wikimedia.org:29418/analytics/wmde/scripts
refs/changes/13/300913/1
diff --git a/src/betafeatures/counts.php b/src/betafeatures/counts.php
index 76b06b2..5543c5b 100755
--- a/src/betafeatures/counts.php
+++ b/src/betafeatures/counts.php
@@ -89,6 +89,11 @@
Output::timestampedMessage( "SELECT FROM
staging.wmde_analytics_betafeature_users FAILED!!" );
} else {
foreach( $queryResult as $row ) {
- WikimediaGraphite::sendNow(
'daily.betafeatures.global_user_counts.totals.' . $row['feature'],
$row['count'] );
+ if ( in_array( $row['feature'], $currentFeatures ) &&
$row['count'] > 0 ) {
+ WikimediaGraphite::sendNow(
+ 'daily.betafeatures.global_user_counts.totals.' .
$row['feature'],
+ $row['count']
+ );
+ }
}
}
\ No newline at end of file
--
To view, visit https://gerrit.wikimedia.org/r/300913
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I249f247a7d11b4bd173a08f3e00ff751855d65a6
Gerrit-PatchSet: 1
Gerrit-Project: analytics/wmde/scripts
Gerrit-Branch: production
Gerrit-Owner: Addshore <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
