Maniphest T193170

[reportupdater] eliminate the funnel parameter
Closed, ResolvedPublic
Actions

Assigned To

None

Authored By

	mforns
	Apr 26 2018, 3:02 PM

Description

I think the funnel parameter can be removed entirely.
reportupdater should recognize automatically the type of results and handle them properly.
Also, the funnel parameter has a really bad name, and is confusing.

reportupdater should ignore the funnel attribute and transparently process output files, always applying multidimensional logic.
Remove funnel property from all report config files.
Remove mention of funnel from on-wiki documentation.

Details

Related Changes in Gerrit:

Subject	Repo	Branch	Lines +/-
Remove all instances of deprecated `funnel` attribute	analytics/reportupdater-queries	master	+0 -33
Validate `funnel` deprecation	analytics/reportupdater-queries	master	+0 -1
Always run as "funnel"	analytics/reportupdater	master	+46 -88

Customize query in gerrit

Related Objects
Search...

		Status	Subtype	Assigned	Task
		Declined		None	T193167 reportupdater TLC
		Resolved		None	T193170 [reportupdater] eliminate the funnel parameter

Event Timeline

mforns created this task.Apr 26 2018, 3:02 PM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptApr 26 2018, 3:02 PM

mforns added a parent task: T193167: reportupdater TLC.Apr 26 2018, 3:02 PM

mforns triaged this task as Unbreak Now! priority.Apr 26 2018, 4:32 PM

mforns moved this task from Incoming to Smart Tools for Better Data on the Analytics board.

Restricted Application added subscribers: Liuxinyu970226, TerraCodes. · View Herald TranscriptApr 26 2018, 4:32 PM

mforns lowered the priority of this task from Unbreak Now! to Needs Triage.Apr 26 2018, 4:34 PM

mforns triaged this task as Medium priority.Apr 30 2018, 4:48 PM

• Vvjjkkii renamed this task from [reportupdater] eliminate the funnel parameter to h5daaaaaaa.Jul 1 2018, 1:13 AM

• Vvjjkkii raised the priority of this task from Medium to High.

• Vvjjkkii added projects: CheckUser, Connected-Open-Heritage-Batch-uploads (RAÄ-KMB_1_2017-02), Tamil-Sites, Gamepress, Hashtags, Jade, KartoEditor, Language-2018-Apr-June, New-Editor-Experiences, Mail, TCB-Team (now WMDE-TechWish).

• Vvjjkkii updated the task description. (Show Details)

• Vvjjkkii removed a subscriber: Aklapper.

AfroThundr3007730 renamed this task from h5daaaaaaa to [reportupdater] eliminate the funnel parameter.Jul 1 2018, 7:05 AM

AfroThundr3007730 lowered the priority of this task from High to Medium.

AfroThundr3007730 removed projects: TCB-Team (now WMDE-TechWish), Mail, New-Editor-Experiences, Language-2018-Apr-June, KartoEditor, Jade, Hashtags, Gamepress, Tamil-Sites, Connected-Open-Heritage-Batch-uploads (RAÄ-KMB_1_2017-02), CheckUser.

AfroThundr3007730 updated the task description. (Show Details)

AfroThundr3007730 added a subscriber: Aklapper.

mforns raised the priority of this task from Medium to Needs Triage.Mar 25 2019, 5:33 PM

mforns triaged this task as Medium priority.

mforns lowered the priority of this task from Medium to Lowest.Dec 9 2019, 5:46 PM

Change 667150 had a related patch set uploaded (by Awight; owner: Awight):
[analytics/reportupdater@master] Always run as "funnel"

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

gerritbot added a project: Patch-For-Review.Feb 26 2021, 12:17 PM

awight subscribed.Feb 26 2021, 12:18 PM

awight added projects: WMDE-TechWish-Sprint-2021-02-17, Unplanned-Sprint-Work.Feb 26 2021, 3:36 PM

awight moved this task from Sprint Backlog to Watching (Stalled) on the WMDE-TechWish-Sprint-2021-02-17 board.

Change 667150 merged by jenkins-bot:
[analytics/reportupdater@master] Always run as "funnel"

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

Maintenance_bot removed a project: Patch-For-Review.Feb 26 2021, 5:11 PM

awight moved this task from Watching (Stalled) to Done on the WMDE-TechWish-Sprint-2021-02-17 board.Mar 2 2021, 9:00 AM

Change 676333 had a related patch set uploaded (by Awight; author: Awight):

[analytics/reportupdater-queries@master] Validate `funnel` deprecation

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

gerritbot added a project: Patch-For-Review.Apr 1 2021, 12:17 PM

Change 676333 merged by Mforns:

[analytics/reportupdater-queries@master] Validate `funnel` deprecation

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

Maintenance_bot removed a project: Patch-For-Review.Apr 2 2021, 4:10 PM

awight mentioned this in rARPQ0e6a2a322c71: Validate `funnel` deprecation.Apr 2 2021, 4:47 PM

Change 682702 had a related patch set uploaded (by Awight; author: Awight):

[analytics/reportupdater-queries@master] Remove all instances of deprecated `funnel` attribute

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

gerritbot added a project: Patch-For-Review.Apr 26 2021, 6:17 PM

awight removed a project: Unplanned-Sprint-Work.Apr 26 2021, 6:18 PM

awight updated the task description. (Show Details)

awight updated the task description. (Show Details)Apr 26 2021, 6:20 PM

Change 682702 merged by Mforns:

[analytics/reportupdater-queries@master] Remove all instances of deprecated `funnel` attribute

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

awight mentioned this in rARPQ0cea2269c2ac: Remove all instances of deprecated `funnel` attribute.May 4 2021, 3:52 PM

Maintenance_bot removed a project: Patch-For-Review.May 4 2021, 4:10 PM

awight closed this task as Resolved.May 4 2021, 8:29 PM