Revert "Revert "ref(cache): Move buffering of pending envelope to ProjectCache" (#1906)"

This reverts commit d016d0f.

Author: olksdr

CHANGELOG.md

@@ -22,6 +22,7 @@
 - Revert back the addition of metric names as tag on Sentry errors when relay drops metrics. ([#1873](https://github.com/getsentry/relay/pull/1873))
 - Tag the dynamic sampling decision on `count_per_root_project` to measure effective sample rates. ([#1870](https://github.com/getsentry/relay/pull/1870))
 - Deprecate fields on the profiling sample format. ([#1878](https://github.com/getsentry/relay/pull/1878))
+- Move the pending envelopes buffering into the project cache. ([#1879](https://github.com/getsentry/relay/pull/1879))
 - Remove idle samples at the start and end of a profile and useless metadata. ([#1894](https://github.com/getsentry/relay/pull/1894))
 
 ## 23.2.0

relay-server/src/actors/project.rs

@@ -1,4 +1,3 @@
-use std::collections::VecDeque;
 use std::sync::Arc;
 use std::time::Duration;
 
@@ -20,16 +19,15 @@ use relay_system::BroadcastChannel;
 
 use crate::actors::envelopes::{EnvelopeManager, SendMetrics};
 use crate::actors::outcome::{DiscardReason, Outcome};
-use crate::actors::processor::{EnvelopeProcessor, ProcessEnvelope};
-use crate::actors::project_cache::{
-    AddSamplingState, CheckedEnvelope, ProjectCache, RequestUpdate,
-};
+#[cfg(feature = "processing")]
+use crate::actors::processor::EnvelopeProcessor;
+use crate::actors::project_cache::{CheckedEnvelope, ProjectCache, RequestUpdate};
 use crate::envelope::Envelope;
 use crate::extractors::RequestMeta;
 
 use crate::service::Registry;
 use crate::statsd::RelayCounters;
-use crate::utils::{self, EnvelopeContext, EnvelopeLimiter, MetricsLimiter, RetryBackoff};
+use crate::utils::{EnvelopeContext, EnvelopeLimiter, MetricsLimiter, RetryBackoff};
 
 #[cfg(feature = "processing")]
 use crate::actors::processor::RateLimitFlushBuckets;
@@ -399,8 +397,6 @@ pub struct Project {
     config: Arc<Config>,
     state: Option<Arc<ProjectState>>,
     state_channel: Option<StateChannel>,
-    pending_validations: VecDeque<(Box<Envelope>, EnvelopeContext)>,
-    pending_sampling: VecDeque<ProcessEnvelope>,
     rate_limits: RateLimits,
     last_no_cache: Instant,
 }
@@ -416,8 +412,6 @@ impl Project {
             config,
             state: None,
             state_channel: None,
-            pending_validations: VecDeque::new(),
-            pending_sampling: VecDeque::new(),
             rate_limits: RateLimits::new(),
             last_no_cache: Instant::now(),
         }
@@ -453,7 +447,7 @@ impl Project {
     /// Returns the project state if it is not expired.
     ///
     /// Convenience wrapper around [`expiry_state`](Self::expiry_state).
-    fn valid_state(&self) -> Option<Arc<ProjectState>> {
+    pub fn valid_state(&self) -> Option<Arc<ProjectState>> {
         match self.expiry_state() {
             ExpiryState::Updated(state) => Some(state),
             ExpiryState::Stale(state) => Some(state),
@@ -642,93 +636,9 @@ impl Project {
         self.get_cached_state(no_cache);
     }
 
-    /// Validates the envelope and submits the envelope to the next stage.
-    ///
-    /// If this project is disabled or rate limited, corresponding items are dropped from the
-    /// envelope. Remaining items in the Envelope are forwarded:
-    ///  - If the envelope needs dynamic sampling, this sends [`AddSamplingState`] to the
-    ///    [`ProjectCache`] to add the required project state.
-    ///  - Otherwise, the envelope is directly submitted to the [`EnvelopeProcessor`].
-    fn flush_validation(
-        &mut self,
-        envelope: Box<Envelope>,
-        envelope_context: EnvelopeContext,
-        project_state: Arc<ProjectState>,
-    ) {
-        if let Ok(checked) = self.check_envelope(envelope, envelope_context) {
-            if let Some((envelope, envelope_context)) = checked.envelope {
-                let mut process = ProcessEnvelope {
-                    envelope,
-                    envelope_context,
-                    project_state,
-                    sampling_project_state: None,
-                };
-
-                if let Some(sampling_key) = utils::get_sampling_key(&process.envelope) {
-                    let own_key = process
-                        .project_state
-                        .get_public_key_config()
-                        .map(|c| c.public_key);
-
-                    if Some(sampling_key) == own_key {
-                        process.sampling_project_state = Some(process.project_state.clone());
-                        EnvelopeProcessor::from_registry().send(process);
-                    } else {
-                        ProjectCache::from_registry()
-                            .send(AddSamplingState::new(sampling_key, process));
-                    }
-                } else {
-                    EnvelopeProcessor::from_registry().send(process);
-                }
-            }
-        }
-    }
-
-    /// Enqueues an envelope for validation.
-    ///
-    /// If the project state is up to date, the message will be immediately sent to the next stage.
-    /// Otherwise, this queues the envelope and flushes it when the project has been updated.
-    ///
-    /// This method will trigger an update of the project state internally if the state is stale or
-    /// outdated.
-    pub fn enqueue_validation(&mut self, envelope: Box<Envelope>, context: EnvelopeContext) {
-        match self.get_cached_state(envelope.meta().no_cache()) {
-            Some(state) if !state.invalid() => self.flush_validation(envelope, context, state),
-            _ => self.pending_validations.push_back((envelope, context)),
-        }
-    }
-
-    /// Adds the project state for dynamic sampling and submits the Envelope for processing.
-    fn flush_sampling(&self, mut message: ProcessEnvelope) {
-        // Intentionally ignore all errors. Fallback sampling behavior applies in this case.
-        if let Some(state) = self.valid_state().filter(|state| !state.invalid()) {
-            // Never use rules from another organization.
-            if state.organization_id == message.project_state.organization_id {
-                message.sampling_project_state = Some(state);
-            }
-        }
-
-        EnvelopeProcessor::from_registry().send(message);
-    }
-
-    /// Enqueues an envelope for adding a dynamic sampling project state.
-    ///
-    /// If the project state is up to date, the message will be immediately submitted for
-    /// processing. Otherwise, this queues the envelope and flushes it when the project has been
-    /// updated.
-    ///
-    /// This method will trigger an update of the project state internally if the state is stale or
-    /// outdated.
-    pub fn enqueue_sampling(&mut self, message: ProcessEnvelope) {
-        match self.get_cached_state(message.envelope.meta().no_cache()) {
-            Some(_) => self.flush_sampling(message),
-            None => self.pending_sampling.push_back(message),
-        }
-    }
-
     /// Replaces the internal project state with a new one and triggers pending actions.
     ///
-    /// This flushes pending envelopes from [`ValidateEnvelope`] and [`AddSamplingState`] and
+    /// This flushes pending envelopes from [`ValidateEnvelope`] and
     /// notifies all pending receivers from [`get_state`](Self::get_state).
     ///
     /// `no_cache` should be passed from the requesting call. Updates with `no_cache` will always
@@ -775,16 +685,6 @@ impl Project {
             return;
         }
 
-        // Flush all queued `ValidateEnvelope` messages
-        while let Some((envelope, context)) = self.pending_validations.pop_front() {
-            self.flush_validation(envelope, context, state.clone());
-        }
-
-        // Flush all queued `AddSamplingState` messages
-        while let Some(message) = self.pending_sampling.pop_front() {
-            self.flush_sampling(message);
-        }
-
         // Flush all waiting recipients.
         relay_log::debug!("project state {} updated", self.project_key);
         channel.inner.send(state);
@@ -921,18 +821,6 @@ impl Project {
     }
 }
 
-impl Drop for Project {
-    fn drop(&mut self) {
-        let count = self.pending_validations.len() + self.pending_sampling.len();
-        if count > 0 {
-            relay_log::with_scope(
-                |scope| scope.set_tag("project_key", self.project_key),
-                || relay_log::error!("dropped project with {} envelopes", count),
-            );
-        }
-    }
-}
-
 #[cfg(test)]
 mod tests {
     use std::sync::Arc;