abcxyz / abcxyz/github-metrics-aggregator

Investigate Using Actual Event Times

Ouverte
#139 1 commentaire 0 réactions 0 personnes assignées Voir sur GitHub
enhancement
Langage dominant
Go
Étoiles
16
Forks
3
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

Currently, GMA adds a "received" time to every Big Query row. This time is based on when it receives the webhook, which may not be the same time as the event (especially if there are retries).

This may cause some queries to miss events you would expect in a certain timeframe.

It would be better to extract the event time from the payload of the event, though I haven't seen if every event type keeps that information in the same place.

Its worth investigating feasibility of extracting actual times either when processing the webhook, or as a scheduled query to update rows with a more accurate time, or to copy to a new table.

Guide de contribution

Ouvrir le guide de contribution

Évaluation

Cette issue n'a pas encore été évaluée.

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.