gavioto / rietveld

Automatically exported from code.google.com/p/rietveld
Apache License 2.0
0 stars 0 forks source link

Deadline Exceeded error trying to view http://codereview.chromium.org/7165006/ #312

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
I'm a Googler working on Chromium.  I uploaded a change consisting mainly of an 
"svn propset" on several hundred files, at 
http://codereview.chromium.org/7165006/ .  Trying to view it always results in 
a long wait and then a "deadline exceeded" error traceback.

Original issue reported on code.google.com by pkasting@chromium.org on 14 Jun 2011 at 9:56

GoogleCodeExporter commented 9 years ago

Original comment by john.abd...@gmail.com on 14 Jun 2011 at 9:57

GoogleCodeExporter commented 9 years ago

Original comment by john.abd...@gmail.com on 14 Jun 2011 at 9:58

GoogleCodeExporter commented 9 years ago
Exceeded soft process size limit with 191.352 MB after servicing 380 requests 
total.

Looks like rietveld just uses too much heap on that change. I recommend doing a 
manual review.

Original comment by maruel@chromium.org on 14 Jun 2011 at 10:09

GoogleCodeExporter commented 9 years ago
It seems in principle like it shouldn't require 200 MB+ of memory to generate a 
list of files changed.  I can certainly work around things in this specific 
case, but it'd be nice to find the top memory consumer here and smash it.

Original comment by pkasting@chromium.org on 14 Jun 2011 at 10:12

GoogleCodeExporter commented 9 years ago

Original comment by albrecht.andi on 6 Apr 2012 at 7:41