qe: Fix Windows build after merging c-abi (#4826)

Metrics feature does not really make a lot of sense in that case - all
it does is excluding single property from a struct making our life
harder.

libs/query-engine-common/Cargo.toml

@@ -3,9 +3,6 @@ name = "query-engine-common"
 version = "0.1.0"
 edition = "2021"
 
-[features]
-metrics = []
-
 [dependencies]
 thiserror = "1"
 url.workspace = true

libs/query-engine-common/src/engine.rs

@@ -58,7 +58,6 @@ pub struct EngineBuilder {
 pub struct ConnectedEngineNative {
     pub config_dir: PathBuf,
     pub env: HashMap<String, String>,
-    #[cfg(feature = "metrics")]
     pub metrics: Option<query_engine_metrics::MetricRegistry>,
 }
 

query-engine/query-engine-c-abi/Cargo.toml

@@ -8,9 +8,6 @@ doc = false
 crate-type = ["staticlib"]
 name = "query_engine"
 
-[features]
-metrics = ["query-engine-common/metrics"]
-
 [dependencies]
 anyhow = "1"
 async-trait = "0.1"

query-engine/query-engine-c-abi/src/engine.rs

@@ -266,7 +266,6 @@ impl QueryEngine {
                 native: ConnectedEngineNative {
                     config_dir: builder.native.config_dir.clone(),
                     env: builder.native.env.clone(),
-                    #[cfg(feature = "metrics")]
                     metrics: None,
                 },
             }) as Result<ConnectedEngine>

query-engine/query-engine-node-api/Cargo.toml

@@ -25,9 +25,7 @@ request-handlers = { path = "../request-handlers", features = [
     "all",
 ] }
 query-connector = { path = "../connectors/query-connector" }
-query-engine-common = { path = "../../libs/query-engine-common", features = [
-    "metrics",
-] }
+query-engine-common = { path = "../../libs/query-engine-common" }
 user-facing-errors = { path = "../../libs/user-facing-errors" }
 psl = { workspace = true, features = ["all"] }
 sql-connector = { path = "../connectors/sql-query-connector", package = "sql-query-connector", features = [