refactor: Remove person_overrides from HogQL database schema (#22735)

PostHog · Jun 18, 2024 · f0a8172 · f0a8172
1 parent 59eaa99
commit f0a8172
Show file tree

Hide file tree

Showing 3 changed files with 0 additions and 195 deletions.
diff --git a/posthog/hogql/database/database.py b/posthog/hogql/database/database.py
@@ -45,10 +45,6 @@
     RawPersonDistinctIdsTable,
 )
 from posthog.hogql.database.schema.persons import PersonsTable, RawPersonsTable, join_with_persons_table
-from posthog.hogql.database.schema.person_overrides import (
-    PersonOverridesTable,
-    RawPersonOverridesTable,
-)
 from posthog.hogql.database.schema.session_replay_events import (
     RawSessionReplayEventsTable,
     SessionReplayEventsTable,
@@ -92,7 +88,6 @@ class Database(BaseModel):
     persons: PersonsTable = PersonsTable()
     person_distinct_ids: PersonDistinctIdsTable = PersonDistinctIdsTable()
     person_distinct_id_overrides: PersonDistinctIdOverridesTable = PersonDistinctIdOverridesTable()
-    person_overrides: PersonOverridesTable = PersonOverridesTable()
 
     session_replay_events: SessionReplayEventsTable = SessionReplayEventsTable()
     cohort_people: CohortPeople = CohortPeople()
@@ -109,7 +104,6 @@ class Database(BaseModel):
     raw_groups: RawGroupsTable = RawGroupsTable()
     raw_cohort_people: RawCohortPeople = RawCohortPeople()
     raw_person_distinct_id_overrides: RawPersonDistinctIdOverridesTable = RawPersonDistinctIdOverridesTable()
-    raw_person_overrides: RawPersonOverridesTable = RawPersonOverridesTable()
     raw_sessions: RawSessionsTable = RawSessionsTable()
 
     # system tables
@@ -121,7 +115,6 @@ class Database(BaseModel):
         "groups",
         "persons",
         "person_distinct_ids",
-        "person_overrides",
         "session_replay_events",
         "cohort_people",
         "static_cohort_people",

diff --git a/posthog/hogql/database/schema/person_overrides.py b/posthog/hogql/database/schema/person_overrides.py
diff --git a/posthog/hogql/database/test/__snapshots__/test_database.ambr b/posthog/hogql/database/test/__snapshots__/test_database.ambr
@@ -552,58 +552,6 @@
           "name": "person_distinct_ids",
           "type": "posthog"
       },
-      "person_overrides": {
-          "fields": {
-              "old_person_id": {
-                  "chain": null,
-                  "fields": null,
-                  "hogql_value": "old_person_id",
-                  "name": "old_person_id",
-                  "schema_valid": true,
-                  "table": null,
-                  "type": "string"
-              },
-              "override_person_id": {
-                  "chain": null,
-                  "fields": null,
-                  "hogql_value": "override_person_id",
-                  "name": "override_person_id",
-                  "schema_valid": true,
-                  "table": null,
-                  "type": "string"
-              },
-              "oldest_event": {
-                  "chain": null,
-                  "fields": null,
-                  "hogql_value": "oldest_event",
-                  "name": "oldest_event",
-                  "schema_valid": true,
-                  "table": null,
-                  "type": "datetime"
-              },
-              "merged_at": {
-                  "chain": null,
-                  "fields": null,
-                  "hogql_value": "merged_at",
-                  "name": "merged_at",
-                  "schema_valid": true,
-                  "table": null,
-                  "type": "datetime"
-              },
-              "created_at": {
-                  "chain": null,
-                  "fields": null,
-                  "hogql_value": "created_at",
-                  "name": "created_at",
-                  "schema_valid": true,
-                  "table": null,
-                  "type": "datetime"
-              }
-          },
-          "id": "person_overrides",
-          "name": "person_overrides",
-          "type": "posthog"
-      },
       "session_replay_events": {
           "fields": {
               "session_id": {
@@ -1912,58 +1860,6 @@
           "name": "person_distinct_ids",
           "type": "posthog"
       },
-      "person_overrides": {
-          "fields": {
-              "old_person_id": {
-                  "chain": null,
-                  "fields": null,
-                  "hogql_value": "old_person_id",
-                  "name": "old_person_id",
-                  "schema_valid": true,
-                  "table": null,
-                  "type": "string"
-              },
-              "override_person_id": {
-                  "chain": null,
-                  "fields": null,
-                  "hogql_value": "override_person_id",
-                  "name": "override_person_id",
-                  "schema_valid": true,
-                  "table": null,
-                  "type": "string"
-              },
-              "oldest_event": {
-                  "chain": null,
-                  "fields": null,
-                  "hogql_value": "oldest_event",
-                  "name": "oldest_event",
-                  "schema_valid": true,
-                  "table": null,
-                  "type": "datetime"
-              },
-              "merged_at": {
-                  "chain": null,
-                  "fields": null,
-                  "hogql_value": "merged_at",
-                  "name": "merged_at",
-                  "schema_valid": true,
-                  "table": null,
-                  "type": "datetime"
-              },
-              "created_at": {
-                  "chain": null,
-                  "fields": null,
-                  "hogql_value": "created_at",
-                  "name": "created_at",
-                  "schema_valid": true,
-                  "table": null,
-                  "type": "datetime"
-              }
-          },
-          "id": "person_overrides",
-          "name": "person_overrides",
-          "type": "posthog"
-      },
       "session_replay_events": {
           "fields": {
               "session_id": {