search

abcxyz / github-metrics-aggregator

Apache License 2.0
11 stars 3 forks source link

Fix to only use delivery_id when de-duplicating events #156

Closed pdewilde closed 8 months ago

pdewilde commented 8 months ago

delivery_id is the best source of truth. Also avoids using SELECT DISTINCT in the new table with a JSON type, as DISTINCT isn't supported for JSON.

resolves #130

Kudos to @lock14 for the suggestion

pdewilde commented 8 months ago

Also I'm a big dummy and forgot I hadn't merged in #154 which is why I had so many issues