fix: remove name label from QUERY_DURATION metrics

graphprotocol · Sep 29, 2023 · f851ff1 · f851ff1
1 parent 50fb34e
commit f851ff1
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/service/src/metrics/mod.rs b/service/src/metrics/mod.rs
@@ -90,7 +90,7 @@ pub static QUERY_DURATION: Lazy<HistogramVec> = Lazy::new(|| {
         .namespace("indexer")
         .subsystem("service")
         .buckets(linear_buckets(0.0, 1.0, 20).unwrap()),
-        &["deployment", "name"],
+        &["deployment"],
     )
     .expect("Failed to create query_duration histograms");
     prometheus::register(Box::new(m.clone())).expect("Failed to register query_duration counter");

diff --git a/service/src/server/routes/status.rs b/service/src/server/routes/status.rs
@@ -40,7 +40,7 @@ pub async fn status_queries(
     // Read the requested query string
     let query_string = match String::from_utf8(body_bytes.to_vec()) {
         Ok(s) => s,
-        Err(e) => return bad_request_response(&e.to_string()),
+        Err(e) => bad_request_response(&e.to_string()),
     };
 
     // filter supported root fields
@@ -63,7 +63,7 @@ pub async fn status_queries(
 
     let response: reqwest::Response = match request.send().await {
         Ok(r) => r,
-        Err(e) => return bad_request_response(&e.to_string()),
+        Err(e) => bad_request_response(&e.to_string()),
     };
 
     match response.text().await {