Add TODO message as a reminder to simplify reading span data

Author: narsaynorath

src/sentry/utils/performance_issues/detectors/consecutive_db_detector.py

@@ -216,6 +216,8 @@ def is_event_eligible(cls, event, project: Project = None) -> bool:
 
         if request:
             url = request.get("url", "") or ""
+            # TODO(nar): `method` can be removed once SDK adoption has increased and
+            # we are receiving `http.method` consistently, likely beyond October 2023
             method = request.get("http.method", "") or request.get("method", "") or ""
             if url.endswith("/graphql") and method.lower() in ["post", "get"]:
                 return False

src/sentry/utils/performance_issues/detectors/large_payload_detector.py

@@ -24,6 +24,8 @@ def visit_span(self, span: Span) -> None:
             return
 
         data = span.get("data", None)
+        # TODO(nar): `Encoded Body Size` can be removed once SDK adoption has increased and
+        # we are receiving `http.response_content_length` consistently, likely beyond October 2023
         encoded_body_size = data and (
             data.get("http.response_content_length", None) or data.get("Encoded Body Size")
         )

src/sentry/utils/performance_issues/detectors/render_blocking_asset_span_detector.py

@@ -100,6 +100,8 @@ def _is_blocking_render(self, span):
             return False
 
         minimum_size_bytes = self.settings.get("minimum_size_bytes")
+        # TODO(nar): `Encoded Body Size` can be removed once SDK adoption has increased and
+        # we are receiving `http.response_content_length` consistently, likely beyond October 2023
         encoded_body_size = (
             data
             and (data.get("http.response_content_length", 0) or data.get("Encoded Body Size", 0))

src/sentry/utils/performance_issues/detectors/uncompressed_asset_detector.py

@@ -36,6 +36,8 @@ def visit_span(self, span: Span) -> None:
             return
 
         data = span.get("data", None)
+        # TODO(nar): The sentence-style keys can be removed once SDK adoption has increased and
+        # we are receiving snake_case keys consistently, likely beyond October 2023
         transfer_size = data and (
             data.get("http.transfer_size", None) or data.get("Transfer Size", None)
         )