Browse Source

ref(mep): Rename the dataset to generic_metrics (#36508)

- This is a quick hacky way to update the dataset name, will do a larger
  change later to properly introduce it as a dataset separate from
  "release health" (`metrics` instead of `generic_metrics`)
William Mak 2 years ago
parent
commit
d1786beb17
2 changed files with 10 additions and 4 deletions
  1. 9 3
      src/sentry/search/events/builder.py
  2. 1 1
      tests/sentry/search/events/test_builder.py

+ 9 - 3
src/sentry/search/events/builder.py

@@ -1889,7 +1889,8 @@ class MetricsQueryBuilder(QueryBuilder):
                 )
 
         return Request(
-            dataset=self.dataset.value,
+            # TODO: Actually introduce this as a dataset
+            dataset="generic_metrics" if self.dataset is Dataset.Metrics else self.dataset.value,
             app_id="default",
             query=Query(
                 match=primary_framework.entity,
@@ -2084,7 +2085,10 @@ class MetricsQueryBuilder(QueryBuilder):
                     granularity=self.granularity,
                 )
                 request = Request(
-                    dataset=self.dataset.value,
+                    # TODO: Actually introduce this as a dataset
+                    dataset="generic_metrics"
+                    if self.dataset is Dataset.Metrics
+                    else self.dataset.value,
                     app_id="default",
                     query=query,
                     flags=Flags(turbo=self.turbo),
@@ -2277,7 +2281,9 @@ class TimeseriesMetricQueryBuilder(MetricsQueryBuilder):
             if len(query_details.functions) > 0:
                 queries.append(
                     Request(
-                        dataset=self.dataset.value,
+                        dataset="generic_metrics"
+                        if self.dataset is Dataset.Metrics
+                        else self.dataset.value,
                         app_id="default",
                         query=Query(
                             match=query_details.entity,

+ 1 - 1
tests/sentry/search/events/test_builder.py

@@ -1252,7 +1252,7 @@ class MetricQueryBuilderTest(MetricBuilderBaseTest):
     def test_get_snql_query(self):
         query = MetricsQueryBuilder(self.params, "", selected_columns=["p90(transaction.duration)"])
         snql_request = query.get_snql_query()
-        assert snql_request.dataset == "metrics"
+        assert snql_request.dataset == "generic_metrics"
         snql_query = snql_request.query
         self.assertCountEqual(
             snql_query.select,