Skip to content

Insights/global intervals

Warren Gifford requested to merge insights/global-intervals into main

Created by: coury-clark

Closes #28873 (closed)

Alright, what's going on here.

The functional changes are as follows:

  1. Global series now respect time intervals on the series for both historical and non-historical recordings.
  2. Non-historical recordings will schedule a future execution one interval away from the time they executed last.
  3. Fixing a bug in the frame generation logic (relevant for capture groups insights) and wrapping with tests
  4. Adding more tests for the interval math

Then on top of that, I prepared too much spaghetti and had to break some circular dependencies. Ever since the capture groups work we have had this weird thing where a subpackage queryrunner had type definitions used by a higher level package query, and as soon as I added this into the store package there were numerous cyclical dependencies.

Here is an image of a global insight with a 4 day interval. The last point is much lower due to my local index being rebuilt and matches the search results at the time. image

Merge request reports

Loading