Bearloga has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/373628 )

Change subject: metrics::maps::users_by_country: Fix wmf.webrequest partition
......................................................................


metrics::maps::users_by_country: Fix wmf.webrequest partition

'maps' is no longer its own partition and that data is now in 'uploads'

Bug: T167083
Change-Id: I756194d6fd458f67346992173536167deac731af
---
M modules/metrics/maps/users_by_country
1 file changed, 2 insertions(+), 1 deletion(-)

Approvals:
  Bearloga: Verified; Looks good to me, approved



diff --git a/modules/metrics/maps/users_by_country 
b/modules/metrics/maps/users_by_country
index b0350dd..b01b6ad 100755
--- a/modules/metrics/maps/users_by_country
+++ b/modules/metrics/maps/users_by_country
@@ -9,9 +9,10 @@
            ELSE 'Other' END AS country
     FROM webrequest
     WHERE
-      webrequest_source = 'maps'
+      webrequest_source = 'upload'
       AND CONCAT(year, '-', LPAD(month, 2, '0'), '-', LPAD(day, 2, '0')) >= 
'$1'
       AND CONCAT(year, '-', LPAD(month, 2, '0'), '-', LPAD(day, 2, '0')) < '$2'
+      AND uri_host = 'maps.wikimedia.org'
       AND http_status IN('200','304')
       AND uri_path RLIKE 
'^/([^/]+)/([0-9]{1,2})/(-?[0-9]+)/(-?[0-9]+)(@([0-9]\\.?[0-9]?)x)?\\.([a-z]+)$'
       AND uri_query <> '?loadtesting'

-- 
To view, visit https://gerrit.wikimedia.org/r/373628
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I756194d6fd458f67346992173536167deac731af
Gerrit-PatchSet: 1
Gerrit-Project: wikimedia/discovery/golden
Gerrit-Branch: master
Gerrit-Owner: Bearloga <[email protected]>
Gerrit-Reviewer: Bearloga <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to