Investigate N+1 query issue in ee/spec/requests/api/graphql/group/epics_spec.rb
In ee/spec/requests/api/graphql/group/epics_spec.rb
the test which checks N+1 queries was not accurate and there seems to be still N+1 issue (which the test was checking) - it would be good to investigate the source of the issue and fix it if possible.
The following discussion from !59360 (merged) should be addressed:
-
@jprovaznik started a discussion: (+1 comment) I think this test was not accurate - it used different params for getting control count vs measure count -
iids
param makes the difference because if it's passed then we search for epics only in the existing group. Ifiids
was not passed, then control count was "penalized" because it did extra queries for finding all descendant groups.This provides same query difference with and w/o the optimization in EpicsFinder - it would be better to investigate/fix this in a separate follow-up.