abcxyz / abcxyz/github-metrics-aggregator

Investigate Using Actual Event Times

未关闭
#139 1 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看
enhancement
主要语言
Go
星标
16
派生
3
PR 合并指标
30 天内没有已合并 PR

描述

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.

贡献指南

打开贡献指南

评估

这个 Issue 还没有评估数据。

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。