Skip to content

Commit

Permalink
CP-49158: [later] introduce a batching delay on first response
Browse files Browse the repository at this point in the history
Also introduce a batching delay on all Event.{from,next} calls, except
Event.from calls for tasks. This should help batch up multiple events from
field updates.

There are no extra delays for tasks, because these typically aren't immediately ready,
and when they are we want to immediately find out about it.

This is part of the "feature flag" that turns on the feature
(previously you could've manually edited xapi.conf to make these changes.)

Signed-off-by: Edwin Török <[email protected]>
  • Loading branch information
edwintorok committed Dec 12, 2024
1 parent 8bf9a0d commit 7fccbed
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion ocaml/xapi/xapi_globs.ml
Original file line number Diff line number Diff line change
Expand Up @@ -1097,7 +1097,8 @@ let make_batching name ~delay_before ~delay_between =
(config, (name, Arg.String set, get, desc))

let event_from_delay, event_from_entry =
make_batching "event_from" ~delay_before:Mtime.Span.zero
make_batching "event_from"
~delay_before:Mtime.Span.(50 * ms)
~delay_between:Mtime.Span.(50 * ms)

let event_from_task_delay, event_from_task_entry =
Expand Down

0 comments on commit 7fccbed

Please sign in to comment.