ref: don't use CanonicalKeyView for interfaces (#71099)

these are already canonicalized names

<!-- Describe your PR here. -->

Author: asottile-sentry

src/sentry/eventstore/models.py

@@ -26,7 +26,6 @@
 from sentry.models.event import EventDict
 from sentry.snuba.events import Columns
 from sentry.spans.grouping.api import load_span_grouping_config
-from sentry.utils.canonical import CanonicalKeyView
 from sentry.utils.safe import get_path, trim
 from sentry.utils.strings import truncatechars
 
@@ -70,9 +69,7 @@ def __init__(
     def __getstate__(self) -> Mapping[str, Any]:
         state = self.__dict__.copy()
         # do not pickle cached info.  We want to fetch this on demand
-        # again.  In particular if we were to pickle interfaces we would
-        # pickle a CanonicalKeyView which old sentry workers do not know
-        # about
+        # again.
         state.pop("_project_cache", None)
         state.pop("_environment_cache", None)
         state.pop("_group_cache", None)
@@ -304,12 +301,9 @@ def project(self, project: Project) -> None:
             self.project_id = project.id
         self._project_cache = project
 
-    def get_interfaces(self) -> Mapping[str, Interface]:
-        return CanonicalKeyView(get_interfaces(self.data))
-
     @cached_property
     def interfaces(self) -> Mapping[str, Interface]:
-        return self.get_interfaces()
+        return get_interfaces(self.data)
 
     def get_interface(self, name: str) -> Interface | None:
         return self.interfaces.get(name)