Remove unused graph endpoint

It was created a few years ago, but it wasn't actually used anywhere.

Author: Kobzol

site/src/request_handlers.rs

@@ -9,7 +9,7 @@ mod status_page;
 pub use bootstrap::handle_bootstrap;
 pub use dashboard::handle_dashboard;
 pub use github::handle_github;
-pub use graph::{handle_graph, handle_graphs};
+pub use graph::handle_graphs;
 pub use next_artifact::handle_next_artifact;
 pub use self_profile::{
     handle_self_profile, handle_self_profile_processed_download, handle_self_profile_raw,

site/src/request_handlers/graph.rs

@@ -3,21 +3,12 @@ use std::collections::HashMap;
 use std::sync::Arc;
 
 use crate::api::graphs::GraphKind;
-use crate::api::{graph, graphs, ServerResult};
+use crate::api::{graphs, ServerResult};
 use crate::db::{self, ArtifactId, Profile, Scenario};
 use crate::interpolate::IsInterpolated;
 use crate::load::SiteCtxt;
 use crate::selector::{CompileBenchmarkQuery, CompileTestCase, Selector, SeriesResponse};
 
-pub async fn handle_graph(
-    request: graph::Request,
-    ctxt: Arc<SiteCtxt>,
-) -> ServerResult<graph::Response> {
-    log::info!("handle_graph({:?})", request);
-
-    create_graph(request, ctxt).await
-}
-
 pub async fn handle_graphs(
     request: graphs::Request,
     ctxt: Arc<SiteCtxt>,
@@ -51,31 +42,6 @@ pub async fn handle_graphs(
     Ok(resp)
 }
 
-async fn create_graph(
-    request: graph::Request,
-    ctxt: Arc<SiteCtxt>,
-) -> ServerResult<graph::Response> {
-    let artifact_ids = artifact_ids_for_range(&ctxt, request.start, request.end);
-    let mut series_iterator = ctxt
-        .statistic_series(
-            CompileBenchmarkQuery::default()
-                .benchmark(Selector::One(request.benchmark))
-                .profile(Selector::One(request.profile.parse()?))
-                .scenario(Selector::One(request.scenario.parse()?))
-                .metric(Selector::One(request.metric.parse()?)),
-            Arc::new(artifact_ids),
-        )
-        .await?
-        .into_iter()
-        .map(SeriesResponse::interpolate);
-
-    let result = series_iterator.next().unwrap();
-    let graph_series = graph_series(result.series, request.kind);
-    Ok(graph::Response {
-        series: graph_series,
-    })
-}
-
 async fn create_graphs(
     request: graphs::Request,
     ctxt: &SiteCtxt,
@@ -145,19 +111,6 @@ async fn create_graphs(
     }))
 }
 
-/// Returns artifact IDs for the given range.
-/// Inside of the range (not at the start/end), only master commits are kept.
-fn artifact_ids_for_range(ctxt: &SiteCtxt, start: Bound, end: Bound) -> Vec<ArtifactId> {
-    let range = ctxt.data_range(start..=end);
-    let count = range.len();
-    range
-        .into_iter()
-        .enumerate()
-        .filter(|(index, commit)| *index == 0 || *index == count - 1 || commit.is_master())
-        .map(|c| c.1.into())
-        .collect()
-}
-
 /// Returns master commit artifact IDs for the given range.
 fn master_artifact_ids_for_range(ctxt: &SiteCtxt, start: Bound, end: Bound) -> Vec<ArtifactId> {
     ctxt.data_range(start..=end)

site/src/server.rs

@@ -375,14 +375,6 @@ async fn serve_req(server: Server, req: Request) -> Result<Response, ServerError
                 crate::comparison::handle_triage(input, &ctxt).await,
             ));
         }
-        "/perf/graph" => {
-            let query = check!(parse_query_string(req.uri()));
-            return server
-                .handle_fallible_get_async(&req, &compression, |c| {
-                    request_handlers::handle_graph(query, c)
-                })
-                .await;
-        }
         "/perf/graphs" => {
             let query = check!(parse_query_string(req.uri()));
             return server