Skip to content

Aggregations: add extended timeout feature

Administrator requested to merge insights/aggregation-explicit-timeout into main

Created by: unclejustin

Closes Search aggregation insights do not allow to trigger explicit aggregation when timeout was hit

Design

Screenshot of implementation

PixelSnap 2022-09-01 at 10 26 23

Screenshot of design

PixelSnap 2022-09-01 at 10 27 21

Request timing out in 2 seconds then with 60 second max timeout

PixelSnap 2022-09-01 at 10 14 26

Test plan

  • Open devtools and switch to the "Network" tab
  • Filter for "GetSearchAggregation"
  • Run a search for any term. Use type:diff to make sure it times out (Should timeout in 2 seconds in the network tab)
  • Should display "Run aggregation" over the skeleton chart as seen in the screenshot above
  • Clicking button should re-run aggregation query with a 60s timeout
  • Clicking another aggregation button should maintain the 60s timeout
  • Changing the query should reset timeout to 2s

App preview:

Check out the client app preview documentation to learn more.

Merge request reports

Loading