From aab4f6c65129a22637113737abdb7bfbeef401f6 Mon Sep 17 00:00:00 2001
From: Dom Dwyer <dom@itsallbroken.com>
Date: Mon, 9 Jan 2023 14:28:16 +0100
Subject: [PATCH] refactor: remove unused QueryExec impl

This is completely unused and left over from the initial skeleton.
---
 ingester2/src/query/exec.rs | 40 -------------------------------------
 ingester2/src/query/mod.rs  |  2 +-
 2 files changed, 1 insertion(+), 41 deletions(-)
 delete mode 100644 ingester2/src/query/exec.rs

diff --git a/ingester2/src/query/exec.rs b/ingester2/src/query/exec.rs
deleted file mode 100644
index b0693ea3d0..0000000000
--- a/ingester2/src/query/exec.rs
+++ /dev/null
@@ -1,40 +0,0 @@
-use async_trait::async_trait;
-use data_types::{NamespaceId, TableId};
-use observability_deps::tracing::*;
-use trace::span::{Span, SpanRecorder};
-
-use super::{QueryError, QueryExec};
-use crate::query::response::QueryResponse;
-
-#[derive(Debug)]
-pub(crate) struct QueryRunner;
-
-impl QueryRunner {
-    pub(crate) fn new() -> Self {
-        Self {}
-    }
-}
-
-#[async_trait]
-impl QueryExec for QueryRunner {
-    type Response = QueryResponse;
-
-    async fn query_exec(
-        &self,
-        namespace_id: NamespaceId,
-        table_id: TableId,
-        columns: Vec<String>,
-        span: Option<Span>,
-    ) -> Result<Self::Response, QueryError> {
-        let mut _span_recorder = SpanRecorder::new(span);
-
-        info!(
-            namespace_id=%namespace_id,
-            table_id=%table_id,
-            columns=?columns,
-            "executing query"
-        );
-
-        unimplemented!();
-    }
-}
diff --git a/ingester2/src/query/mod.rs b/ingester2/src/query/mod.rs
index c5d8d2a9d5..14c9b8f7eb 100644
--- a/ingester2/src/query/mod.rs
+++ b/ingester2/src/query/mod.rs
@@ -7,7 +7,7 @@ pub(crate) use r#trait::*;
 pub(crate) mod partition_response;
 pub(crate) mod response;
 
-pub(crate) mod exec;
+// Instrumentation
 pub(crate) mod instrumentation;
 pub(crate) mod tracing;