From aaf11036529460cc80c5ae3373cc3ae4f36d5e64 Mon Sep 17 00:00:00 2001
From: "J. Nick Koston" <nick@koston.org>
Date: Fri, 10 Mar 2023 09:45:24 -1000
Subject: [PATCH] fix stale docs strings, remove dead code

---
 homeassistant/components/recorder/queries.py  |  9 ---------
 .../recorder/table_managers/event_types.py    | 20 +++++++++----------
 2 files changed, 10 insertions(+), 19 deletions(-)

diff --git a/homeassistant/components/recorder/queries.py b/homeassistant/components/recorder/queries.py
index b0e0e7a8900..5a558fc6208 100644
--- a/homeassistant/components/recorder/queries.py
+++ b/homeassistant/components/recorder/queries.py
@@ -50,15 +50,6 @@ def get_shared_event_datas(hashes: list[int]) -> StatementLambdaElement:
     )
 
 
-def find_event_type_id(event_type: str) -> StatementLambdaElement:
-    """Find an event_type id by event_type."""
-    return lambda_stmt(
-        lambda: select(EventTypes.event_type_id).filter(
-            EventTypes.event_type == event_type
-        )
-    )
-
-
 def find_event_type_ids(event_types: Iterable[str]) -> StatementLambdaElement:
     """Find an event_type id by event_type."""
     return lambda_stmt(
diff --git a/homeassistant/components/recorder/table_managers/event_types.py b/homeassistant/components/recorder/table_managers/event_types.py
index 1a2dfbbf10f..aaa13074530 100644
--- a/homeassistant/components/recorder/table_managers/event_types.py
+++ b/homeassistant/components/recorder/table_managers/event_types.py
@@ -16,29 +16,29 @@ CACHE_SIZE = 2048
 
 
 class EventTypeManager:
-    """Manage event types."""
+    """Manage the EventTypes table."""
 
     def __init__(self) -> None:
-        """Initialize the event manager."""
+        """Initialize the event type manager."""
         self._id_map: dict[str, int] = LRU(CACHE_SIZE)
         self._pending: dict[str, EventTypes] = {}
         self.active = False
 
     def load(self, events: list[Event], session: Session) -> None:
-        """Load the event types into memory."""
+        """Load the event_type to event_type_ids mapping into memory."""
         self.get_many(
             (event.event_type for event in events if event.event_type is not None),
             session,
         )
 
     def get(self, event_type: str, session: Session) -> int | None:
-        """Resolve events to event data."""
+        """Resolve event_type to the event_type_id."""
         return self.get_many((event_type,), session)[event_type]
 
     def get_many(
         self, event_types: Iterable[str], session: Session
     ) -> dict[str, int | None]:
-        """Resolve events to event data."""
+        """Resolve event_types to event_type_ids."""
         results: dict[str, int | None] = {}
         missing: list[str] = []
         for event_type in event_types:
@@ -61,27 +61,27 @@ class EventTypeManager:
         return results
 
     def get_pending(self, event_type: str) -> EventTypes | None:
-        """Get pending event type."""
+        """Get pending EventTypes that have not be assigned ids yet."""
         return self._pending.get(event_type)
 
     def add_pending(self, db_event_type: EventTypes) -> None:
-        """Add a pending event."""
+        """Add a pending EventTypes that will be committed at the next interval."""
         assert db_event_type.event_type is not None
         event_type: str = db_event_type.event_type
         self._pending[event_type] = db_event_type
 
     def post_commit_pending(self) -> None:
-        """Flush pending events."""
+        """Call after commit to load the event_type_ids of the new EventTypes into the LRU."""
         for event_type, db_event_types in self._pending.items():
             self._id_map[event_type] = db_event_types.event_type_id
         self._pending.clear()
 
     def reset(self) -> None:
-        """Reset the event manager."""
+        """Reset the event manager after the database has been reset or changed."""
         self._id_map.clear()
         self._pending.clear()
 
     def evict_purged(self, event_types: Iterable[str]) -> None:
-        """Evict purged event types."""
+        """Evict purged event_types from the cache when they are no longer used."""
         for event_type in event_types:
             self._id_map.pop(event_type, None)