Refactor Editor Analytics according to the new AQSAssist
Open, HighPublic3 Estimated Story Points
Actions

Assigned To

Authored By

	JEbe-WMF
	Mar 20 2024, 2:13 PM

Description

Goal

Extract to aqsassist some repeated and common code that is present in all the services and refactor them to avoid code duplication as we already did with the functions we created to validate the parameters in the handlers.

About the Druid common code refactoring we will take the following into consideration:

The way to parameterize Druid queries that was implemented in AQS 1.0 (https://github.com/wikimedia/analytics-aqs/blob/master/sys/mediawiki-history-schemas.yaml) to parametrize Druid queries using a separate file
We'll start refactoring editor-analytics because its code is already done and we'll have the opportunity of using its unit and integration tests to test the refactored service.

Tasks

Refactor
QA test
Deploy

Details

	Subject	Repo	Branch	Lines +/-
	Refactor Editor Analytics with AQS-Assist	generated-data-platform/aqs/editor-analytics	main	+194 -215

Customize query in gerrit

Related Objects
Search...

		Status	Subtype	Assigned	Task
		Open		JEbe-WMF	T343907 AQS 2.0: aqsassist - Extract some repeated common code from services to this module
		Open		EChukwukere-WMF	T360528 Refactor Editor Analytics according to the new AQSAssist

Event Timeline

JEbe-WMF created this task.Mar 20 2024, 2:13 PM

Restricted Application removed a project: Patch-For-Review. · View Herald TranscriptMar 20 2024, 2:13 PM

JEbe-WMF moved this task from Sprint Backlog to In Process on the Data Products (Data Products Sprint 10) board.Mar 20 2024, 2:14 PM

WDoranWMF moved this task from In Process to Protal to Sprint 11 on the Data Products (Data Products Sprint 10) board.Mar 21 2024, 2:03 PM

WDoranWMF edited projects, added Data Products (Data Products Sprint 11); removed Data Products (Data Products Sprint 10).

WDoranWMF moved this task from Sprint Backlog to In Process on the Data Products (Data Products Sprint 11) board.

JEbe-WMF set the point value for this task to 3.Mar 21 2024, 6:17 PM

Change #1013548 had a related patch set uploaded (by Jennifer Ebe; author: Jennifer Ebe):

[generated-data-platform/aqs/editor-analytics@main] Refactor Editor Analytics with AQS-Assist

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

gerritbot added a project: Patch-For-Review.Mar 22 2024, 2:21 PM

Sfaci moved this task from In Process to Code Review / Tech Input on the Data Products (Data Products Sprint 11) board.Mar 25 2024, 6:29 PM

Sfaci moved this task from Code Review / Tech Input to In Process on the Data Products (Data Products Sprint 11) board.

Sfaci moved this task from In Process to Code Review / Tech Input on the Data Products (Data Products Sprint 11) board.Mar 26 2024, 10:07 AM

VirginiaPoundstone triaged this task as Low priority.Mar 29 2024, 4:26 PM

SGupta-WMF claimed this task.Apr 1 2024, 2:26 AM

SGupta-WMF reassigned this task from SGupta-WMF to JEbe-WMF.Apr 4 2024, 11:07 AM

JEbe-WMF moved this task from Code Review / Tech Input to Testing on the Data Products (Data Products Sprint 11) board.Apr 17 2024, 10:01 AM

JEbe-WMF edited projects, added Data Products (Data Products Sprint 12); removed Data Products (Data Products Sprint 11).Apr 17 2024, 2:10 PM

JEbe-WMF moved this task from Sprint Backlog to Code Review / Tech Input on the Data Products (Data Products Sprint 12) board.Apr 17 2024, 2:12 PM

JEbe-WMF moved this task from Code Review / Tech Input to Testing on the Data Products (Data Products Sprint 12) board.

JEbe-WMF reassigned this task from JEbe-WMF to EChukwukere-WMF.Apr 17 2024, 2:16 PM

Change #1013548 merged by jenkins-bot:

[generated-data-platform/aqs/editor-analytics@main] Refactor Editor Analytics with AQS-Assist

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

The changes here have been merged to prod. This has been tested through regression run for Editor analytics and no issues found. All tests passed

EChukwukere-WMF moved this task from Testing to To Deploy on the Data Products (Data Products Sprint 12) board.Apr 24 2024, 3:07 PM

xcollazo moved this task from To Deploy to Testing on the Data Products (Data Products Sprint 12) board.Apr 24 2024, 4:27 PM

Pppery removed a project: Patch-For-Review.Apr 30 2024, 4:12 AM

Test status: QA PASS

PROD url to test QA env with, aggregate : 'https://wikimedia.org/api/rest_v1/metrics/editors/aggregate/sw.wikipedia/all-editor-types/all-page-types/all-activity-levels/daily/20190101/20190102'

PROD url to test QA env with, editors_top_by_absolute_bytes_diff: 'https://wikimedia.org/api/rest_v1/metrics/editors/top-by-absolute-bytes-difference/sw.wikipedia/all-editor-types/all-page-types/2019/01/01'

Local env version, aggregate: 'http://localhost:8094/metrics/editors/aggregate/sw.wikipedia/all-editor-types/all-page-types/all-activity-levels/daily/20190101/20190102'

Local env version, editors_top_by_absolute_bytes_diff : 'http://localhost:8094/metrics/editors/top-by-absolute-bytes-difference/sw.wikipedia/all-editor-types/all-page-types/2019/01/01'

data from prod and local were semantically identical.

No regression issues and ready for deployment.

EChukwukere-WMF moved this task from Testing to To Deploy on the Data Products (Data Products Sprint 12) board.Apr 30 2024, 5:38 PM

VirginiaPoundstone edited projects, added Data Products (Data Products Sprint 13); removed Data Products (Data Products Sprint 12), good first task, AQS2.0.May 1 2024, 10:13 PM

VirginiaPoundstone moved this task from Sprint Backlog to To Deploy on the Data Products (Data Products Sprint 13) board.May 1 2024, 10:14 PM

Sfaci reassigned this task from JEbe-WMF to EChukwukere-WMF.Wed, May 29, 12:21 PM

Sfaci edited projects, added Data Products (Data Products Sprint 14), AQS2.0; removed Data Products (Data Products Sprint 13).

Sfaci moved this task from Sprint Backlog to Testing on the Data Products (Data Products Sprint 14) board.Wed, May 29, 12:25 PM

VirginiaPoundstone raised the priority of this task from Low to High.Wed, May 29, 4:31 PM

Test status in PROD for Editor Analysis : QA PASS

There were slight changes made to validation errors that I needed minor test update

4 features passed, 0 failed, 24 skipped
73 scenarios passed, 0 failed, 1157 skipped
710 steps passed, 0 failed, 9941 skipped, 0 undefined
Took 0m6.616s

EChukwukere-WMF moved this task from Testing to To Deploy on the Data Products (Data Products Sprint 14) board.Wed, May 29, 6:05 PM

EChukwukere-WMF moved this task from To Deploy to Done on the Data Products (Data Products Sprint 14) board.Wed, May 29, 7:38 PM

Refactor Editor Analytics according to the new AQSAssistOpen, HighPublic3 Estimated Story PointsActions