Versions Compared

Key

  • This line was added.
  • This line was removed.
  • Formatting was changed.

...

    Although only present on the larger courses, a significant amount of 500s were being returned. A solution has been proposed to include an index to account for this but for now in order to continue the load tests, the offending parameters "last_activity_at" and "asc" will be removed from out random selection. This also means that our thread retrieval for PATCHing (and DELETEing which has been disabled) will be less random, but sufficiently random. By removing these parameters, we hope to get rid of these 500s and see a performance improvement for large and small courses.

Test reruns:

    These tests have the non-indexed fields removed just to verify that those were causing the 500s.

Expand

 

Image Added

Namereqs#failsAvgMinMaxMedianreq/s95%
GETGET_comment_list2620(0.00%)1881486441800.2260
GETGET_thread167780(0.00%)185137120118017.6240
GETGET_thread_list91250(0.00%)417140142641010750
PATCHPATCH_comment2601(0.38%)34817916813000660
PATCHPATCH_thread3263(0.91%)2481467132200.1440
Expand

Image Added

 

NEXT STEPS:

Address the 500s

...