abcxyz / abcxyz/github-metrics-aggregator
Investigate Using Actual Event Times
- 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
Bewertung
Dieses Issue wurde noch nicht bewertet.