Issue metadata
Sign in to add a comment
|
Gerrit is very slow |
||||||||||||||||||||||||
Issue descriptionLoading codereview page and going to the next file takes non-trivial amount of time for me
,
Aug 26 2017
I just opened CL from email, clicked on the first file and it tool like 8 seconds to load this page for me: https://chromium-review.googlesource.com/c/chromium/src/+/627598/4/ios/chrome/test/earl_grey/chrome_earl_grey.h
,
Aug 28 2017
,
Aug 28 2017
Is it possible to report on the overall performance dashboard/trend of gerrit so we know if the slowness reported here is a one-time thing or a general problem for all gerrit users?
,
Aug 28 2017
Yes, we track outages in chopsdash.appspot.com, and you can see any graph you care to name at http://vi/gerritcodereview/gerrit.
,
Aug 28 2017
Please avoid putting .corp URLs in external bugs.
,
Aug 28 2017
Sorry. Looking at the dashboard, it appears that there was indeed an increase in latency around middle of the day of August 24 (which coincided with the report date of this crbug).
,
Sep 8 2017
Once again it took ~5 seconds to fully load the page. After reloading I've got blank page and then had to wait until it's content is loaded. Do you have any monitoring tools to measure load latency? Do you think it's only a problem on my end? I don't want to reopen the bug again, but Geritt quite often is slow for me. I never had similar issues with Rietveld.
,
Sep 11 2017
The GoB team is working very hard on three classes of performance problems: slow loading of Gerrit pages, slow fetches (especially of refs/changes/* refs), and slow pushes (especially to refs/changes/* refs). All of these problems are interconnected, and are due to the combination of the large size of our repository and the large rate of creation of changes refs. (Incidentally, Android is way worse off than we are on all of these metrics, so that's fun.) All of these are being tracked in b/ bugs by the GoB team.
,
Sep 11 2017
Thank you for the detailed answer.
,
Nov 13 2017
It has been 2 month and performance has not improved. 5 seconds is enough for me to get distracted and start working on something else which significantly affects my productivity. I do not agree that closing this bug is a reasonable resolution. External Dependency or Duplicate can be a valid status, but not Fixed. Julie, is it possible to prioritize Gerrit performance work?
,
Nov 14 2017
My understanding is that this *is* the top priority for Gerrit team. Aaron, Shawn, Logan - can you confirm?
,
Nov 14 2017
Shawn is in the best position to provide details about priority, progress, and timelines. But I can say that I have been reassured by Shawn multiple times that performance is the Gerrit backend team's top priority.
,
Nov 14 2017
Thank you for the update. Do you have references to b/ bugs mentioned in comment #10?
,
Nov 14 2017
b/35998605 is the internal master bug that we believe root causes the performance problems. Its being worked on full time by engineers, and has been for months. Unfortunately its a difficult backend data migration that must take place to correct the problem. We've been uncovering some "unknown unknowns" related to how to do this safely without breaking the entire system and causing a work stoppage for our customers. Resolving these and iterating has been slowing progress. We are also subject to some rollout speed concerns; we only get 1 release per week. If we uncover something about a new release on Monday, we have to wait a full 7 days before we can resolve whatever that was and try again. That has been severely slowing our calendar. If it were possible to iterate faster, we would uncover these "unknown unknowns" more rapidly. As I told Julie, this is a top priority to my team. We are working on it.
,
Jan 3 2018
|
|||||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||||
Comment 1 by katthomas@google.com
, Aug 25 2017