Skip to content

Commit 82e97fb

Browse files
authored
chore(data-warehouse): clarify webhook delivery-status INSERT helper comment
The INSERT helper is not referenced by any test; correct the misleading "used by tests" comment to describe it as a bypass-Kafka producer helper kept for parity with the hog_invocation_results table family. Generated-By: PostHog Code Task-Id: 77e746c7-ae01-4e1a-9621-076bb86deb00
1 parent b534fea commit 82e97fb

1 file changed

Lines changed: 4 additions & 2 deletions

File tree

  • posthog/models/warehouse_webhook_delivery_status

posthog/models/warehouse_webhook_delivery_status/sql.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -131,8 +131,10 @@ def WAREHOUSE_WEBHOOK_DELIVERY_STATUS_ENGINE() -> MergeTreeEngine:
131131
)
132132

133133

134-
# Direct insert used by tests / any bypass-Kafka producer. Writes go to the
135-
# local data table (the distributed read alias isn't writable).
134+
# Direct insert helper for a bypass-Kafka producer (e.g. a backfill or future
135+
# test fixture). Writes go to the local data table (the distributed read alias
136+
# isn't writable). Kept alongside the DROP/TRUNCATE helpers for parity with the
137+
# `hog_invocation_results` table family.
136138
INSERT_WAREHOUSE_WEBHOOK_DELIVERY_STATUS_SQL = f"""
137139
INSERT INTO {WAREHOUSE_WEBHOOK_DELIVERY_STATUS_DATA_TABLE} (
138140
team_id,

0 commit comments

Comments
 (0)