Gilles has submitted this change and it was merged.
Change subject: Add missing date limit to one of the action UNION ALLs
......................................................................
Add missing date limit to one of the action UNION ALLs
Change-Id: I5809c44d1ddcfd698949b7ba6ad429cc3102f9b5
---
M actions/template.sql
1 file changed, 2 insertions(+), 1 deletion(-)
Approvals:
Gilles: Verified; Looks good to me, approved
diff --git a/actions/template.sql b/actions/template.sql
index e040ac6..b485422 100644
--- a/actions/template.sql
+++ b/actions/template.sql
@@ -41,7 +41,8 @@
SELECT timestamp, event_action, event_samplingFactor FROM
MediaViewer_8935662
WHERE %wiki% timestamp < TIMESTAMP(CURDATE()) AND
timestamp >= TIMESTAMP(DATE_SUB(CURDATE(), INTERVAL 30 DAY))
UNION ALL
- SELECT timestamp, (CASE WHEN event_value = 0 THEN
'pref-optout-loggedin' ELSE 'pref-optin-loggedin' END) AS event_action, 1 AS
event_samplingFactor FROM PrefUpdate_5563398 WHERE event_property =
'multimediaviewer-enable'
+ SELECT timestamp, (CASE WHEN event_value = 0 THEN
'pref-optout-loggedin' ELSE 'pref-optin-loggedin' END) AS event_action, 1 AS
event_samplingFactor FROM PrefUpdate_5563398
+ WHERE event_property = 'multimediaviewer-enable' AND
timestamp < TIMESTAMP(CURDATE()) AND timestamp >= TIMESTAMP(DATE_SUB(CURDATE(),
INTERVAL 30 DAY))
) AS MediaViewerUnioned
GROUP BY datestring
--
To view, visit https://gerrit.wikimedia.org/r/148518
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I5809c44d1ddcfd698949b7ba6ad429cc3102f9b5
Gerrit-PatchSet: 1
Gerrit-Project: analytics/multimedia
Gerrit-Branch: master
Gerrit-Owner: Gergő Tisza <[email protected]>
Gerrit-Reviewer: Gilles <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
