Add support for X-WMF-UUID method of transmitting app install ID to apps uniques reports [5 pts] {hawk}
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	• Deskana
	Apr 22 2015, 10:04 PM

Description

Originally, the app install ID was only transmitted as a query parameter. This was problematic for caching. For a while, we've been transmitting it both as a query parameter and in the X-Analytics headers, as a transition. We're now preparing to remove it from the query parameter, and only transmit the install ID via the X-Analytics header.

We'd like the apps uniques reports and session reports to be updated to use the X-Analytics header as well as the query parameter.

The reports will need to continue to support the old method using the query parameter because people that don't update their apps will continue to send their install ID that way.

The header documentation is here, under "wmf-uuid": https://wikitech.wikimedia.org/wiki/X-Analytics

Details

	Subject	Repo	Branch	Lines +/-
	Use wmfuuid from x_analytics_map for app install id if available else default to the query parameter	analytics/refinery	master	+8 -4

Customize query in gerrit

Related Objects

Mentioned In: rANRE4e4d456e7e90: Use wmfuuid from x_analytics_map for app install id if available else default…
T96927: Evaluate citoid for citations used on Wikimedia sites
T92985: Onboard Madhu
Mentioned Here: T96927: Evaluate citoid for citations used on Wikimedia sites

Event Timeline

• Deskana created this task.Apr 22 2015, 10:04 PM

• Deskana raised the priority of this task from to Needs Triage.

• Deskana updated the task description. (Show Details)

• Deskana added a project: Analytics.

• Deskana subscribed.

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptApr 22 2015, 10:04 PM

Session code is taken care of, as it never used the app install id from url. Changes will be on this patch: https://gerrit.wikimedia.org/r/#/c/199935/ (still WIP)

The app uniques report however does need updating, both daily and monthly.

• Nuria added projects: Analytics-Clusters, Analytics-Kanban.Apr 22 2015, 10:36 PM

• Nuria set Security to None.

• Nuria mentioned this in T92985: Onboard Madhu.

• Nuria added a subscriber: • kevinator.

• Deskana renamed this task from Add support for X-WMF-UUID method of transmitting app install ID to apps uniques and session reports to Add support for X-WMF-UUID method of transmitting app install ID to apps uniques reports.Apr 22 2015, 10:37 PM

current query looks at URL:
https://github.com/wikimedia/analytics-refinery/blob/master/oozie/mobile_apps/uniques/daily/generate_uniques_daily.hql

• kevinator renamed this task from Add support for X-WMF-UUID method of transmitting app install ID to apps uniques reports to Add support for X-WMF-UUID method of transmitting app install ID to apps uniques reports [5 pts].Apr 23 2015, 3:15 PM

• kevinator renamed this task from Add support for X-WMF-UUID method of transmitting app install ID to apps uniques reports [5 pts] to Add support for X-WMF-UUID method of transmitting app install ID to apps uniques reports [5 pts] {hawk}.Apr 23 2015, 9:38 PM

• kevinator moved this task from Next Up to Tasked_Hidden on the Analytics-Kanban board.

• madhuvishy claimed this task.Apr 24 2015, 12:04 AM

• kevinator triaged this task as Medium priority.Apr 27 2015, 4:49 PM

• madhuvishy moved this task from Tasked_Hidden to In Code Review on the Analytics-Kanban board.Apr 30 2015, 12:46 AM

Change 207689 had a related patch set uploaded (by Guillom):
[WIP] Use wmfuuid from x_analytics_map for app install id if available else default to the query parameter Todo - test if it works

https://gerrit.wikimedia.org/r/207689

gerritbot added a project: Patch-For-Review.Apr 30 2015, 8:14 PM

Note: The patch was done by @madhuvishy but incorrectly referenced T96927; I simply fixed the commit message.

gpaumier mentioned this in T96927: Evaluate citoid for citations used on Wikimedia sites.Apr 30 2015, 8:20 PM

• kevinator merged a task: T95606: Upgrade unique mobile jobs to harvest unique id from x-analytics field .May 4 2015, 5:40 PM

Change 207689 merged by Joal:
Use wmfuuid from x_analytics_map for app install id if available else default to the query parameter

https://gerrit.wikimedia.org/r/207689

• madhuvishy mentioned this in rANRE4e4d456e7e90: Use wmfuuid from x_analytics_map for app install id if available else default….May 7 2015, 10:42 AM

• ggellerman moved this task from In Code Review to Ready to Deploy on the Analytics-Kanban board.May 7 2015, 3:44 PM

• ggellerman moved this task from Ready to Deploy to Done on the Analytics-Kanban board.May 12 2015, 3:42 PM

• ggellerman moved this task from Done to Ready to Deploy on the Analytics-Kanban board.May 15 2015, 3:42 PM

This change is done and deployed.