Revert "feat(issue-stream): Combined endpoint for stats + unhandled (#58195)"

This reverts commit 253ef49670f07b7f21562a393c98e4f46c7fb674.

Co-authored-by: roggenkemper <46740234+roggenkemper@users.noreply.github.com>

src/sentry/api/serializers/models/group.py

@@ -25,7 +25,7 @@ import sentry_sdk
 from django.conf import settings
 from django.db.models import Min, prefetch_related_objects
 
-from sentry import features, tagstore
+from sentry import tagstore
 from sentry.api.serializers import Serializer, register, serialize
 from sentry.api.serializers.models.actor import ActorSerializer
 from sentry.api.serializers.models.plugin import is_plugin_deprecated
@@ -493,13 +493,7 @@ class GroupSerializerBase(Serializer, ABC):
     def _get_group_snuba_stats(
         self, item_list: Sequence[Group], seen_stats: Optional[Mapping[Group, SeenStats]]
     ):
-        if (
-            self._collapse("unhandled")
-            and len(item_list) > 0
-            and features.has(
-                "organizations:issue-stream-performance", item_list[0].project.organization
-            )
-        ):
+        if self._collapse("unhandled"):
             return None
         start = self._get_start_from_seen_stats(seen_stats)
         unhandled = {}

src/sentry/api/serializers/models/group_stream.py

@@ -8,7 +8,7 @@ from typing import Any, Callable, Mapping, MutableMapping, Optional, Sequence
 
 from django.utils import timezone
 
-from sentry import features, release_health, tsdb
+from sentry import release_health, tsdb
 from sentry.api.serializers.models.group import (
     BaseGroupSerializerResponse,
     GroupSerializer,
@@ -213,21 +213,10 @@ class StreamGroupSerializerSnuba(GroupSerializerSnuba, GroupStatsMixin):
             attrs = super().get_attrs(item_list, user)
         else:
             seen_stats = self._get_seen_stats(item_list, user)
-
             if seen_stats:
                 attrs = {item: seen_stats.get(item, {}) for item in item_list}
             else:
                 attrs = {item: {} for item in item_list}
-            if len(item_list) > 0 and features.has(
-                "organizations:issue-stream-performance", item_list[0].project.organization
-            ):
-                unhandled_stats = self._get_group_snuba_stats(item_list, seen_stats)
-
-                if unhandled_stats is not None:
-                    for item in item_list:
-                        attrs[item]["is_unhandled"] = bool(
-                            unhandled_stats.get(item.id, {}).get("unhandled")
-                        )
 
         if self.stats_period and not self._collapse("stats"):
             partial_get_stats = functools.partial(
@@ -317,8 +306,6 @@ class StreamGroupSerializerSnuba(GroupSerializerSnuba, GroupStatsMixin):
             }
             if "times_seen" in attrs:
                 result.update(self._convert_seen_stats(attrs))
-            if "is_unhandled" in attrs:
-                result["isUnhandled"] = attrs["is_unhandled"]
 
         if not self._collapse("stats"):
             if self.stats_period:

tests/snuba/api/endpoints/test_organization_group_index_stats.py

@@ -3,7 +3,7 @@ import uuid
 from sentry.issues.grouptype import ProfileFileIOGroupType
 from sentry.issues.occurrence_consumer import process_event_and_issue_occurrence
 from sentry.testutils.cases import APITestCase, SnubaTestCase
-from sentry.testutils.helpers import parse_link_header, with_feature
+from sentry.testutils.helpers import parse_link_header
 from sentry.testutils.helpers.datetime import before_now, iso_format
 from sentry.testutils.silo import region_silo_test
 from tests.sentry.issues.test_utils import OccurrenceTestMixin
@@ -67,36 +67,6 @@ class GroupListTest(APITestCase, SnubaTestCase, OccurrenceTestMixin):
         assert "userCount" in response_data[0]
         assert "lifetime" in response_data[0]
         assert "filtered" in response_data[0]
-        assert "isUnhandled" not in response_data[0]
-
-    @with_feature("organizations:issue-stream-performance")
-    def test_unhandled(self):
-        self.store_event(
-            data={"timestamp": iso_format(before_now(seconds=500)), "fingerprint": ["group-1"]},
-            project_id=self.project.id,
-        )
-        group_a = self.store_event(
-            data={"timestamp": iso_format(before_now(seconds=1)), "fingerprint": ["group-a"]},
-            project_id=self.project.id,
-        ).group
-
-        self.login_as(user=self.user)
-        response = self.get_response(query="is:unresolved", groups=[group_a.id])
-
-        response_data = sorted(response.data, key=lambda x: x["firstSeen"], reverse=True)
-
-        assert response.status_code == 200
-        assert len(response_data) == 1
-        assert "title" not in response_data[0]
-        assert "hasSeen" not in response_data[0]
-        assert "stats" in response_data[0]
-        assert "firstSeen" in response_data[0]
-        assert "lastSeen" in response_data[0]
-        assert "count" in response_data[0]
-        assert "userCount" in response_data[0]
-        assert "lifetime" in response_data[0]
-        assert "filtered" in response_data[0]
-        assert "isUnhandled" in response_data[0]
 
     def test_issue_platform_issue(self):
         event_id = uuid.uuid4().hex