fix: Put query metrics behind feature flag

joshua-spacetime · joshua-spacetime · commit b8799aa68a81 · 2024-02-01T12:09:46.000-08:00
A call site for record_query_duration_metrics was missed in #749.
diff --git a/crates/core/src/subscription/subscription.rs b/crates/core/src/subscription/subscription.rs
@@ -329,6 +329,7 @@ impl QuerySet {
                     }
                 }
             }
+            #[cfg(feature = "metrics")]
             record_query_duration_metrics(WorkloadType::Update, &db.address(), start);
         }
         for (table_id, (table_name, ops)) in table_ops.into_iter().filter(|(_, (_, ops))| !ops.is_empty()) {

Original file line number	Diff line number	Diff line change
`@@ -329,6 +329,7 @@ impl QuerySet {`
`329`	`329`	`}`
`330`	`330`	`}`
`331`	`331`	`}`
	`332`	`+ #[cfg(feature = "metrics")]`
`332`	`333`	`record_query_duration_metrics(WorkloadType::Update, &db.address(), start);`
`333`	`334`	`}`
`334`	`335`	`for (table_id, (table_name, ops)) in table_ops.into_iter().filter(\|(_, (_, ops))\| !ops.is_empty()) {`