Page MenuHomePhabricator

AQS 2.0: Editor Analytics: Create and initialize gerrit repository
Closed, ResolvedPublic

Description

Create gerrit repository for the Editor Analytics service, using the other AQS 2.0 service repositories as an example. Populate with the Go service scaffolding as a starting point, with boilerplate suitably modified as necessary.

Completion criteria: the repository is ready to accept merge requests for endpoint and unit test implementation.

  • repository is created
  • boilerplate is cleaned up

Event Timeline

Change 922163 had a related patch set uploaded (by Santiago Faci; author: Santiago Faci):

[generated-data-platform/aqs/editor-analytics@main] [DNM] First approach for a Druid-base AQS 2.0 endpoint

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

Previous patch is not definitive. It has been made to be able to review some changes to start discussing about the right way to deal with Druid and create reusable code for the two services that use this database engine

Change 922163 abandoned by Santiago Faci:

[generated-data-platform/aqs/editor-analytics@main] [DNM] First approach for a Druid-base AQS 2.0 endpoint

Reason:

no longer needed. About to push a new valid MR

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

Change 922163 restored by Santiago Faci:

[generated-data-platform/aqs/editor-analytics@main] [DNM] First approach for a Druid-base AQS 2.0 endpoint

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

Change 922163 merged by BPirkle:

[generated-data-platform/aqs/editor-analytics@main] First approach for a Druid-base AQS 2.0 endpoint

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

Change merged with the understanding (per separate discussion) that this code is somewhat speculative and significant work remains to make it production ready. But this is a great start!

QA: don't worry about integration tests (make itest) or hitting the endpoint in your browser - you'd need new data in the Druid mock dataset and that's not ready yet. It is sufficient to confirm that these succeed:

  • make check
  • make
  • make test
Sfaci changed the task status from Open to In Progress.May 24 2023, 4:22 PM
Sfaci claimed this task.
BPirkle moved this task from AQS 2.0 Backlog to Sprint 10 on the AQS2.0 board.
BPirkle edited projects, added AQS2.0 (Sprint 10); removed AQS2.0.
BPirkle moved this task from Next Up to Ready for Testing on the AQS2.0 (Sprint 10) board.