Smarter refreshing of authorized projects
This MR reworks the way User#refresh_authorized_projects, resulting in a much more efficient process leading to fewer dead tuples.
Related issue: https://gitlab.com/gitlab-org/gitlab-ce/issues/25257
Merge request reports
Activity
mentioned in issue #25257 (closed)
assigned to @DouweM
@yorickpeterse Douwe is out this week! Can someone else review this, or are you happy to wait until next week?
@yorickpeterse Douwe is out this week! Can someone else review this, or are you happy to wait until next week?
@smcgivern This should go in 8.15 but the merge window closes this Friday, so ideally this week (unless we still merge things next week)?
I think we can merge things next week, but it's better to look earlier.
@nick.thomas / @grzesiek - could one of you review this please?
assigned to @yorickpeterse
added 795 commits
-
48fa5b9c...a50cd9eb - 794 commits from branch
master - d3621578 - Smarter refreshing of authorized projects
-
48fa5b9c...a50cd9eb - 794 commits from branch
assigned to @nick.thomas
assigned to @yorickpeterse
@yorickpeterse a lot of headscratching and false starts and "ahhh, that is OK", and a couple of tiny comments. I think this will be good to merge after they're addressed
👍 assigned to @nick.thomas
mentioned in commit 1139da27
mentioned in commit 896f09b9
