Comments

crates/symbolicator/src/service.rs

@@ -404,6 +404,8 @@ impl RequestService {
             }
         }
 
+        // Using `task_name` as the tag should be fine, there is only a small
+        // fixed number of them.
         metric!(counter("requests.accepted") += 1, "task" => task_name);
 
         let request_id = RequestId::new(uuid::Uuid::new_v4());
@@ -479,6 +481,9 @@ impl RequestService {
             // We stop counting the request as an in-flight request at this point, even though
             // it will stay in the `requests` map for another 90s.
             current_requests.fetch_sub(1, Ordering::Relaxed);
+
+            // Using `task_name` as the tag should be fine, there is only a small
+            // fixed number of them.
             metric!(counter("requests.processed") += 1, "task" => task_name);
 
             // Wait before removing the channel from the computation map to allow clients to