Koulio / rietveld

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

Posting a medium sized diff (~ 10,400 lines) gives me a 500 error #68

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?

I am currently working in issue 8656. For some strange reason when i upload
a new patch of about 10,000 lines I get a 500 error. Link:
http://codereview.appspot.com/8656/show
The diff is 376kb in size (attached).

This error eventually goes away but it takes about 5-10 minutes of reloading.

What is the expected output? What do you see instead?

As with any 500, it would be nicer if it could be avoided. 

Original issue reported on code.google.com by api.jhar...@gtempaccount.com on 21 Nov 2008 at 4:34

Attachments:

GoogleCodeExporter commented 9 years ago
I tried this a second time after realizing that the first time, not only did I 
get a
500 error but also only parts of my diff were actually uploaded. Got a 500 the 
second
time also. Maybe I should split my file into smaller chunks...
- J

Original comment by api.jhar...@gtempaccount.com on 21 Nov 2008 at 4:41

GoogleCodeExporter commented 9 years ago
I didn't think so at first but I guess my upload could have taken longer than 
the
documented cut-off but it seemed like I was getting this error a few seconds 
after
posting my patch. Now when I review the patch it looks like only a part of it 
was
applied: http://codereview.appspot.com/8656/diff2/801:911/1875 (notice the 
cut-off
after line 83)
- J

Original comment by api.jhar...@gtempaccount.com on 21 Nov 2008 at 4:40

GoogleCodeExporter commented 9 years ago
Just tried it again, this time uploading ~5000 lines only and I get a 500 error 
about
2 seconds after I hit submit.
- J

Original comment by api.jhar...@gtempaccount.com on 21 Nov 2008 at 4:45

GoogleCodeExporter commented 9 years ago

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

GoogleCodeExporter commented 9 years ago

Original comment by jrobbins@google.com on 11 Jul 2014 at 6:32