Maniphest T366343

SUP: Loosen fields definition of update event schema
Open, MediumPublic
Actions

Assigned To

None

Authored By

	pfischer
	Fri, May 31, 9:05 AM

Tags

Referenced Files

None

Subscribers

Description

As of now, the SUP does not leverage fat update events. Hence, there is no need for for a strict [[ https://schema.wikimedia.org/repositories/primary/jsonschema/development/cirrussearch/update_pipeline/update/latest.yaml | fields ]] property.

AC:

replace fields property with a less explicit definition
make sure org.wikimedia.discovery.cirrus.updater.common.model.UpdateEventTypeInfo works with the generic schema
make sure org.wikimedia.discovery.cirrus.updater.common.model.FetchFailureEncoder works with the generic schema

Event Timeline

pfischer created this task.Fri, May 31, 9:05 AM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptFri, May 31, 9:05 AM

• OKJ04 added a subtask: T366433: CentralAuth tests broken unless you run them inside Quibble.Mon, Jun 3, 1:39 AM

JJMC89 removed a subtask: T366433: CentralAuth tests broken unless you run them inside Quibble.Mon, Jun 3, 1:44 AM

Gehel triaged this task as Medium priority.Mon, Jun 10, 3:47 PM

Gehel moved this task from needs triage to ML & Data Pipeline on the Discovery-Search board.