1648: post-review throws an exception for Perforce changesets with huge descriptions.

jtsh****@gmai***** (Google Code) (Is this you? Claim this profile.)
March 21, 2011
1481
What version are you running?
1.0.6 

What's the URL of the page containing the problem?
path:/r/<review id>/,

What steps will reproduce the problem?
Just have a huge change description with tons of testing details.

What is the expected output? What do you see instead?
post-review to generate a smart error telling users how to fix their change
description.

What operating system are you using? What browser?
Linux

Please provide any additional information below.

Added Christian's note here for your reference.

>they probably put in a huge Description or Testing field again
>we serialize a change descrption in the database, but if it's too long,
the DB can truncate it
>really need to just artificially enforce a limit..
>this is a database limit
>if you can file a bug against RB, I can look into some ways to be smarter
about it
david
#1 david
  • +Component-RBTools