From aa7a34ffe8b9572c44b4bd36c30f7cf3805a9ed7 Mon Sep 17 00:00:00 2001
From: Tamo <tamo@meilisearch.com>
Date: Thu, 17 Oct 2024 00:43:34 +0200
Subject: [PATCH] make the aggregate method send

---
 meilisearch/src/analytics/mod.rs               | 2 +-
 meilisearch/src/analytics/segment_analytics.rs | 5 ++++-
 2 files changed, 5 insertions(+), 2 deletions(-)

diff --git a/meilisearch/src/analytics/mod.rs b/meilisearch/src/analytics/mod.rs
index 8a0a68bad..f8a589901 100644
--- a/meilisearch/src/analytics/mod.rs
+++ b/meilisearch/src/analytics/mod.rs
@@ -110,7 +110,7 @@ mopafy!(Aggregate);
 
 /// Helper trait to define multiple aggregate with the same content but a different name.
 /// Commonly used when you must aggregate a search with POST or with GET for example.
-pub trait AggregateMethod: 'static + Default {
+pub trait AggregateMethod: 'static + Default + Send {
     fn event_name() -> &'static str;
 }
 
diff --git a/meilisearch/src/analytics/segment_analytics.rs b/meilisearch/src/analytics/segment_analytics.rs
index 1a1bb9226..92f03e48e 100644
--- a/meilisearch/src/analytics/segment_analytics.rs
+++ b/meilisearch/src/analytics/segment_analytics.rs
@@ -72,9 +72,12 @@ pub fn extract_user_agents(request: &HttpRequest) -> Vec<String> {
 }
 
 pub struct Message {
+    // Since the type_id is solved statically we cannot retrieve it from the Box.
+    // Thus we have to send it in the message directly.
     type_id: TypeId,
-    event: Box<dyn Aggregate>,
+    // Same for the aggregate function.
     aggregator_function: fn(Box<dyn Aggregate>, Box<dyn Aggregate>) -> Option<Box<dyn Aggregate>>,
+    event: Box<dyn Aggregate>,
 }
 
 impl Message {