Skip to content

Fix N+1 when indexing data

Terri Chu requested to merge tchu-fix-n+1-processbookkeepingservice into master

What does this MR do and why?

Related to #426110 (closed) An N+1 was observed when debugging a recent incident when indexing data for search into Elasticsearch.

This MR:

  • adds a missing N+1 test for milestones, fixes the N+1 discovered
  • slight change to existing epic N+1 test, fixes the N+1 discovered
  • create new preload scopes to avoid rubocop disable

Screenshots or screen recordings

N/A

How to set up and validate locally

N/A

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Terri Chu

Merge request reports