Sample the CommitterFullError a bit more (#34602)

Don't love this sampling logic directly reading short_msg, but I think it'll serve the
goal here. CommitterFullError is the top error going to sentry,
it's triggering our spike protection, and it's not particularly
useful at high volume.

GitOrigin-RevId: d6e37b379321c799ffba6c1d626d07816a14b56d

Author: nipunn1313

crates/errors/src/lib.rs

@@ -457,7 +457,7 @@ impl ErrorMetadata {
             | ErrorCode::RejectedBeforeExecution
             | ErrorCode::OperationalInternalServerError => Some((sentry::Level::Warning, None)),
 
-            // 1% sampling for OCC/Overloaded/RateLimited, since we only really care about the
+            // Sampling for OCC/Overloaded/RateLimited, since we only really care about the
             // details if they happen at high volume.
             ErrorCode::RateLimited => Some((sentry::Level::Info, Some(0.001))),
             ErrorCode::OCC {
@@ -466,6 +466,9 @@ impl ErrorMetadata {
             ErrorCode::OCC {
                 is_system: true, ..
             } => Some((sentry::Level::Warning, Some(0.01))),
+            ErrorCode::Overloaded if self.short_msg == "CommitterFullError" => {
+                Some((sentry::Level::Warning, Some(0.001)))
+            },
             // we want to see these a bit more than the others above
             ErrorCode::Overloaded => Some((sentry::Level::Warning, Some(0.1))),
         }