feat(alerts): Load best effort data for spans dataset (#88912)

Span alerts should always use the best effort method of querying. This
ensures that the query will be guaranteed to finish as well as showing
the most accurate data possible.

`EventsRequest` just needs to pass down the `sampling` mode prop. Since
the `doEventsRequest` method that gets called downstream already accepts
`sampling` as an argument and passes it to the request, I only need to
add this at the top level.

Author: narsaynorath

Diff for: static/app/components/charts/eventsRequest.tsx

@@ -26,6 +26,7 @@ import type {AggregationOutputType} from 'sentry/utils/discover/fields';
 import {getAggregateAlias, stripEquationPrefix} from 'sentry/utils/discover/fields';
 import type {DiscoverDatasets} from 'sentry/utils/discover/types';
 import type {QueryBatching} from 'sentry/utils/performance/contexts/genericQueryBatcher';
+import type {SamplingMode} from 'sentry/views/explore/hooks/useProgressiveQuery';
 
 export type TimeSeriesData = {
   allTimeseriesData?: EventsStatsData;
@@ -199,6 +200,10 @@ type EventsRequestPartialProps = {
    * Sample rate used for data extrapolation in OnDemandMetricsRequest
    */
   sampleRate?: number;
+  /**
+   * The type of sampling mode used for EAP dataset requests
+   */
+  sampling?: SamplingMode;
   /**
    * Should loading be shown.
    */

Diff for: static/app/views/alerts/rules/metric/triggers/chart/index.spec.tsx

@@ -10,6 +10,7 @@ import {
   AlertRuleThresholdType,
   Dataset,
 } from 'sentry/views/alerts/rules/metric/types';
+import {SAMPLING_MODE} from 'sentry/views/explore/hooks/useProgressiveQuery';
 
 const theme = ThemeFixture();
 
@@ -253,4 +254,53 @@ describe('Incident Rules Create', () => {
       })
     );
   });
+
+  it('uses best effort sampling for span alerts', async () => {
+    const {organization, project, router} = initializeOrg({
+      organization: {features: ['visibility-explore-progressive-loading']},
+    });
+
+    render(
+      <TriggersChart
+        theme={theme}
+        api={api}
+        location={router.location}
+        organization={organization}
+        projects={[project]}
+        query=""
+        timeWindow={1}
+        aggregate="count(span.duration)"
+        dataset={Dataset.EVENTS_ANALYTICS_PLATFORM}
+        triggers={[]}
+        environment={null}
+        comparisonType={AlertRuleComparisonType.COUNT}
+        resolveThreshold={null}
+        thresholdType={AlertRuleThresholdType.BELOW}
+        newAlertOrQuery
+        onDataLoaded={() => {}}
+        isQueryValid
+        showTotalCount
+      />
+    );
+
+    expect(await screen.findByTestId('area-chart')).toBeInTheDocument();
+    expect(await screen.findByTestId('alert-total-events')).toBeInTheDocument();
+
+    expect(eventStatsMock).toHaveBeenCalledWith(
+      expect.anything(),
+      expect.objectContaining({
+        query: {
+          interval: '1m',
+          project: [2],
+          query: '',
+          statsPeriod: '14d',
+          yAxis: 'count(span.duration)',
+          referrer: 'api.organization-event-stats',
+          dataset: 'spans',
+          sampling: SAMPLING_MODE.BEST_EFFORT,
+          useRpc: '1',
+        },
+      })
+    );
+  });
 });

Diff for: static/app/views/alerts/rules/metric/triggers/chart/index.tsx

@@ -59,6 +59,7 @@ import {getComparisonMarkLines} from 'sentry/views/alerts/utils/getComparisonMar
 import {AlertWizardAlertNames} from 'sentry/views/alerts/wizard/options';
 import {getAlertTypeFromAggregateDataset} from 'sentry/views/alerts/wizard/utils';
 import {ConfidenceFooter} from 'sentry/views/explore/charts/confidenceFooter';
+import {SAMPLING_MODE} from 'sentry/views/explore/hooks/useProgressiveQuery';
 
 import type {MetricRule, Trigger} from '../../types';
 import {
@@ -747,7 +748,20 @@ class TriggersChart extends PureComponent<Props, State> {
             {noop}
           </EventsRequest>
         ) : null}
-        <EventsRequest {...baseProps} period={period} dataLoadedCallback={onDataLoaded}>
+        <EventsRequest
+          {...baseProps}
+          period={period}
+          dataLoadedCallback={onDataLoaded}
+          // Span alerts only need to do a best effort request and do not need
+          // preflight requests. A user needs to see the highest fidelity data possible
+          // to set up the alert.
+          sampling={
+            organization.features.includes('visibility-explore-progressive-loading') &&
+            dataset === Dataset.EVENTS_ANALYTICS_PLATFORM
+              ? SAMPLING_MODE.BEST_EFFORT
+              : undefined
+          }
+        >
           {({
             loading,
             errored,