Skip to content

Commit

Permalink
also handle the non force case
Browse files Browse the repository at this point in the history
  • Loading branch information
c-cube committed Jul 22, 2024
1 parent 592814d commit 5d5d909
Showing 1 changed file with 6 additions and 3 deletions.
9 changes: 6 additions & 3 deletions src/client-ocurl/opentelemetry_client_ocurl.ml
Original file line number Diff line number Diff line change
Expand Up @@ -289,8 +289,8 @@ end = struct

let batch_max_size_ = 200

let should_send_batch_ ~config ~now (b : _ Batch.t) : bool =
Batch.len b > 0
let should_send_batch_ ?(side = []) ~config ~now (b : _ Batch.t) : bool =
(Batch.len b > 0 || side != [])
&& (Batch.len b >= batch_max_size_
||
let timeout = Mtime.Span.(config.Config.batch_timeout_ms * ms) in
Expand Down Expand Up @@ -355,7 +355,10 @@ end = struct
if Batch.len batches.traces > 0 then send_traces ()
) else (
let now = Mtime_clock.now () in
if should_send_batch_ ~config ~now batches.metrics then
if
should_send_batch_ ~config ~now batches.metrics
~side:(AList.get gc_metrics)
then
send_metrics ();

if should_send_batch_ ~config ~now batches.traces then send_traces ();
Expand Down

0 comments on commit 5d5d909

Please sign in to comment.