fix: do not emit empty batches

imandra-ai · Jul 6, 2022 · 41e9962 · 41e9962
1 parent 16667a3
commit 41e9962
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/src/client/opentelemetry_client_ocurl.ml b/src/client/opentelemetry_client_ocurl.ml
@@ -240,13 +240,12 @@ end = struct
 
   let pop_if_ready ?(force = false) ~now (self : _ t) : _ list option =
     let@ () = with_mutex_ self.lock in
-    if
-      (force && not (is_empty_ self))
-      || is_full_ self || timeout_expired_ ~now self
+    if self.size > 0 && (force || is_full_ self || timeout_expired_ ~now self)
     then (
       let l = self.q in
       self.q <- [];
       self.size <- 0;
+      assert (l <> []);
       Some l
     ) else
       None