Auto-fix duplicate reviews.

Review Request #898 — Created June 17, 2009 and submitted — Latest diff uploaded

Information

Review Board SVN (deprecated)
trunk

Reviewers

Auto-fix duplicate reviews.

Under times of heavy server load, it's possible to get duplicate draft
Review objects in the database. This is harder with RC3, but still
possible. All it requires is that the databsae is under load and two
comments or reviews are created within a span of time on the server by
the same person on the same review request. This had to be manually
fixed in the database before, and caused frustration for those who hit
it.

This problem is pretty specific to draft reviews. This change now
introduces a central function to call to get the draft review for a
review request, and if more than one review is found, it will
consolidate them into a single review and return it.
This consolidation code has been pretty well tested on an actual live
server with many duplicate reviews. There's also unit tests to make sure
that this functionality continues to work.
    Loading...