Skip to content

Remove user from explore/groups queries

Robert May requested to merge generic-explore-groups into master

What does this MR do and why?

Describe in detail what your merge request does and why.

Should solve the performance issues on the /explore/groups endpoint on gitlab.com. Currently there's a massive speed difference between that page loaded anonymously and as a user, but as it supposedly just lists "public" projects anyway I don't see why there should be a difference, especially as it being slower makes it less usable in the first place.

Curiously this still passes the existing specs, so I haven't added made any changes to those 🤷 This is currently marked as an experiment in the feature flag as the performance issue only really shows up on .com and so I need to see its impact in production.

Related #358944 (closed)

Screenshots or screen recordings

Screenshots are required for UI changes, and strongly recommended for all other merge requests.

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

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 Robert May

Merge request reports

Loading