fix(batch-exports): drop internal inserted_at column from s3 exports (#…

…21191)
PostHog · Mar 28, 2024 · 29f8c26 · 29f8c26
1 parent 617dc39
commit 29f8c26
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 3 deletions.
diff --git a/posthog/temporal/batch_exports/s3_batch_export.py b/posthog/temporal/batch_exports/s3_batch_export.py
@@ -409,8 +409,6 @@ def s3_default_fields() -> list[BatchExportField]:
     batch_export_fields.append({"expression": "elements_chain", "alias": "elements_chain"})
     batch_export_fields.append({"expression": "nullIf(person_properties, '')", "alias": "person_properties"})
     batch_export_fields.append({"expression": "toString(person_id)", "alias": "person_id"})
-    # In contrast to other destinations, we do export this field.
-    batch_export_fields.append({"expression": "COALESCE(inserted_at, _timestamp)", "alias": "inserted_at"})
 
     # Again, in contrast to other destinations, and for historical reasons, we do not include these fields.
     not_exported_by_default = {"team_id", "set", "set_once"}

diff --git a/posthog/temporal/tests/batch_exports/test_s3_batch_export_workflow.py b/posthog/temporal/tests/batch_exports/test_s3_batch_export_workflow.py
@@ -308,7 +308,6 @@ async def assert_clickhouse_records_in_s3(
     {
         "fields": [
             {"expression": "event", "alias": "my_event_name"},
-            {"expression": "inserted_at", "alias": "inserted_at"},
             {"expression": "1 + 1", "alias": "two"},
         ],
         "values": {},