From 316a0b7d58a41f164df19c7cf158ca388b8a0119 Mon Sep 17 00:00:00 2001 From: timedout Date: Thu, 28 May 2026 18:46:27 +0100 Subject: [PATCH] fix: Don't repeat already-included metadata in fetch_state instrument --- src/service/rooms/event_handler/fetch_and_handle_outliers.rs | 2 +- src/service/rooms/event_handler/fetch_state.rs | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/service/rooms/event_handler/fetch_and_handle_outliers.rs b/src/service/rooms/event_handler/fetch_and_handle_outliers.rs index 287ee9553..c7faa14fb 100644 --- a/src/service/rooms/event_handler/fetch_and_handle_outliers.rs +++ b/src/service/rooms/event_handler/fetch_and_handle_outliers.rs @@ -29,7 +29,7 @@ const GET_MISSING_EVENTS_MAX_BATCH_SIZE: u16 = 50; // matches src/server/get_mis /// dependencies, however it is ultimately the sender's responsibility to send /// them in a processable order, so this is just a best effort attempt. It does /// not account for power levels or other tie breaks. -pub async fn build_local_dag( +pub async fn build_local_dag( pdu_map: &HashMap, ) -> conduwuit::Result> { debug_assert!(pdu_map.len() >= 2, "needless call to build_local_dag with less than 2 PDUs"); diff --git a/src/service/rooms/event_handler/fetch_state.rs b/src/service/rooms/event_handler/fetch_state.rs index ea7ed3c3b..f4e19fe96 100644 --- a/src/service/rooms/event_handler/fetch_state.rs +++ b/src/service/rooms/event_handler/fetch_state.rs @@ -22,7 +22,7 @@ impl super::Service { /// /// The end result is a result containing a map of shortstatekeys to event /// IDs. The underlying `Option` is always `Some`. - #[tracing::instrument(skip(self, create_event))] + #[tracing::instrument(skip_all)] pub(super) async fn fetch_state( &self, origin: &ServerName,