abcxyz / abcxyz/github-metrics-aggregator

Investigate Using Actual Event Times

Offen
#139 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
enhancement
Vorherrschende Sprache
Go
Sterne
16
Forks
3
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

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.

Beitragsleitfaden

Beitragsleitfaden öffnen

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.