HelixObs / herald / 26931844952

85%

Build Type

push

github

Committed by Chitrang Patel

Commit Message

fix: dedup repeated helix.* events to prevent DB blowup on infinite retry

Pipelines retrying the same failed operation flood entity_events with
identical rows — same entity ID and same error message — because
WriteEntityEvent was a plain INSERT with no deduplication.

Introduce entity_event_dedup (regular table, not a hypertable) with
PRIMARY KEY (entity_id, event_name, normalised_message). The normalised
message uses fingerprint.Normalise — same volatile-token stripping as the
notification system — so path/ID variants of the same error collapse to
one row. WriteEntityEvent now upserts into this table on every call
(incrementing occurrence_count, refreshing last_seen_at) and writes to
entity_events only on the first occurrence. QueryAlerts is updated to
read occurrence counts from entity_event_dedup.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Coverage Stats

23 of 38 new or added lines in 1 file covered. (60.53%)

2179 of 2558 relevant lines covered (85.18%)

1856.62 hits per line