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
What version are you running?

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

What operating system are you using? What browser?

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
#1 david
  • +Component-RBTools