What's New in Review Board

We've just taken one big step toward our 1.0 tonight with the release of Review Board 1.0 beta 1. This is an important release, and introduces new features, many performance enhancements, and a whole lot of bug fixes.

Two of the big highlights include the ability to disable user registration and some major performance enhancements.

The review request page now handles large numbers of reviews and comments in a much more elegant way. Previously, we'd fetch a diff fragment per comment from the server, which mean that if there were 100 comments, we'd fetch 100 diff fragments, slowing down loading. Now, we're only performing one fetch per reviewed file, no matter how many comments are on that file.

The other major improvement is in the diff viewer. Some deep profiling work was done on the comment and diff loading code in the diff viewer, and we've managed to speed things up significantly. In one of our tests, a large file with only 15 comments would take over 1 second of browser time to load and place comments on the diff. With the new improvements, that has dropped to about 150 milliseconds.

A full summary of the changes are below:

• Fixed index errors shown during some interdiffs and empty diffs. (Bug #892)
• Interdiffs would break when one revision had a newly added file. (Bug #847)
• Interdiffs no longer pre-load all files on a page before showing anything. They now load each file independently, like normal diffs.
• We no longer display the "Download Diff" button on interdiffs. (Bug #873)
• Clicking the "ghost" comment flag in the diff viewer wasn't triggering a comment dialog for that line.
• We now correctly parse Git diffs that don't include "a/" and "b/" prefixes in the filenames. Patch by Andrey Fedorov. (Bug #883)
• We now forward on the Perforce Empty Changeset error, instead of returning an Internal Server Error.
• Fixed some problems with Perforce errors on newer versions of the P4 API, and fixed some issues where we were sometimes stripping valuable data from an error message. (Bug #710)
• Fixed an Internal Server Error when showing the review dialog with a comment made on a screenshot with no caption.
• The "Ship It" label in the review dialog is now associated with the checkbox, meaning the label can be clicked to check/uncheck "Ship It." (Bug #922)
• For LDAP backends, we now log on as an anonymous user before searching for the real user to log in as. This fixes some breakages with people encountered. Patch by Alexey Morozov.
• Changed the default LDAP scope to SUBTREE, in order to fix authentication on some LDAP servers. (Bug #959)
• We now use the review request's draft summary, if set, on the screenshot detail page. (Bug #981)
• HTML tags are no longer interpreted in the Description and Testing Done fields. (Bug #894)
• The "branch" field in change descriptions no longer escape entities (such as arrows). (Bug #898)
• Fixed some XHTML validation errors in the dashboard. Patch by Onkar Shinde. (Bug #932)
• We no longer require logging in for read-only API calls, if anonymous access is enabled on the site. (Bug #767)
• Fixed some small breakages with Django 1.1.

• Added a faster, more scalable version of diff fragment loading. We used to make a call to the server for each comment shown in a review, which became very slow on large reviews. We now only make one call per file, regardless of how many comments are made against that file. This speeds up load times of the review page considerably. (Bug #906)
• Greatly improved performance in the diff viewer. Loading comments and placing them in the diff viewer wasn't nearly as fast as it could have been. Due to these new improvements, a sample diff with 15 comments that used to take over 1 second now takes ~150ms to load. Other improvements have also been made, and should make a noticeable difference in large deployments.
• Improved accessibility for inline editors on the review request page. The "edit" icons can now be activated using the Enter key. Patch by Brad Taylor. (Bug #965)
• Added a new icon for the "My Comments" column in the dashboard that's used when you've specifically posted a review marked "Ship It." (Bug #909)
• Change descriptions are now a lot more readable for the Description and Testing Done fields. (Bug #913)
• Added a dependency on Pygments, in order to install it by default.

• Added support for disabling user registration. This provides a checkbox in the General Settings page for disabling user registration. This only takes effect when using standard authentication, and will hide the "Register" link and redirect requests made to /account/register/ to the /account/login/ page instead.
• Added new administration, user documentation, and a FAQ. This is available on http://www.review-board.org/docs/, and is also available in the reviewboard/docs/ directory.

Some last-minute brokenness snuck into alpha 3, so we're releasing a quick alpha 4 that should fix it. If you installed alpha 3, upgrade immediately.

• post-review should now work with SVN regardless of the locale set. (Bug #888)
• Fixed an infinite loop in the Git diff parser. Patch by Dennis Schön. (Bug #887)
• The first file in the diff viewer would sometimes come back as escaped HTML. It's now embedded correctly.
• The comment dialog's "dirty" state wasn't being reset for new comments. This would leave reviewers unable to make new comments.

Review Board 1.0 alpha 3 has just been released. This release fixes a few major bugs introduced by 1.0 alpha 2 (over-aggressive caching primarily) and several browser-compatibility problems. Safari and Internet Explorer both had compatibility issues with some of the features we introduced, and we've since worked around them.

To upgrade from a previous alpha, just run:

$ sudo easy_install -U ReviewBoard
$ rb-site upgrade /path/to/site

A full summary of the changes are below:

• Generated settings_local.py files no longer have DEBUG=True enabled by default.
• The generated FastCGI and Apache config files should now work better out of the box. We had path issues in reviewboard.fcgi. We also now set the PYTHON_EGG_CACHE directory and default to processing .htaccess files in htdocs (which some distros don't have enabled by default).

• Fixed some major performance issues when typing in the diff viewer. This was the cause of many reports of slowdown and visual oddities when creating comments.
• Unchanged files should no longer show errors in interdiffs.
• Fixed file-based navigation ("<" and ">" keys) in the diff viewer. (Bug #696)
• We no longer show a nonsensical error when SVN logins fail. (Bug #721)
• Fixed anchors in the diff viewer. (Bug #828)
• Fixed the "Expand/Collapse Diffs" toggle in the diff viewer. (Bug #862)

• The review request page is no longer cached across login sessions. We were over-aggressive in the caching and are now a lot more careful.
• Upload Diff/Screenshot dialogs on Safari are now centered instead of being docked to the far-left side.
• Uploading diffs and screenshots now works on Safari.
• Empty Changeset errors in Perforce should now properly be shown to the user. They were in some cases triggering a server-side error.
• The Change Description field no longer animates open on page load.
• Sentences containing the word "bug" or "issue" no longer generate links unless followed by either specifically a number or a "#" followed by alphanumeric characters.
• Fixed some escaping issues in the review request summary, which would cause characters (such as quotes) to appear incorrectly. (Bug #860)
• Fixed some issues with auto-linking text in the Description and Testing Done fields. There were issues on initial page loads with linking URLs that were followed by quotes, "<" and ">". (Bug #864)

• We now pre-load the first file on a page in the diff viewer, in order to make it appear that the page is loading faster. Many people felt the new, faster diff viewer was slower because even after the initial page load, they had to wait to see any diffs. They're now able to work with the first diff the moment the diff viewer loads.
• Added hover/clicked styling for the review request action buttons.

We've just put out a release of Review Board 1.0 alpha 2. This release has some major improvements and bug fixes over alpha 1, and we recommend that everybody upgrades.

The main improvements are performance-related. We've added aggressive browser-side caching support, as well as reorganized load times in our pages in order to reduce load times. The diff viewer now loads almost instantly and then progressively loads diffs on the fly, meaning you can start reviewing right away.

If you have mod_expires and mod_deflate (Apache) or mod_gzip (lighttpd) enabled, you'll see the cache and load time improvements. Apache users will get the new configuration for taking advantage of this for free, but if you're a lighttpd user, you'll need to update your configuration based on our lighttpd.conf template.

Going forward, upgrades using easy_install will not grab nightlies. They will only be grabbing stable releases. If you want to continue using nightlies, please refer to the Getting Started guide.

A full summary of the changes in this release are below:

Bugs Fixed in post-review:

• We now strip the filename coming in from Perforce. There can be a trailing newline sometimes, resulting in a failed upload. (Bug #726)
• We set the correct locale now when invoking svn. Patch by Łukasz Jernaś. (Bug #822)
• Fixed backwards version comparison logic in post-review's GitClient. Patch by Stacey Sheldon.
• Added support for hgsubversion. Patch by Augie Fackler
• Added support for Clearcase. Patch by Daniel Belz.

Bugs Fixed in rb-site:

• rb-site now sets "postgresql_psycopg2" as the DATABASE_ENGINE for PostgreSQL installs. Patch by Rusty Burchfield. (Bug #822)
• rb-site should now generate config files that work correctly on Windows servers.

Bugs Fixed in Review Board:

• Whitespace from usernames are now stripped on LDAP accounts, which prevents duplicate user accounts in the database. (Bug #728)
• Fixed a bug where anchors in comment flags were incorrect and were getting lost when setting the comment count for the flag. This was breaking anchors to parts of the diff. (Bug #820)
• The fields in the comment dialog are now disabled if the user is not logged in. (Bug #827)
• Key presses in dialogs or in the review request details box no longer trigger navigation in the diff viewer. (Bug #828)
• The "Review" link on a review request is no longer displayed if the user is not logged in.
• Trailing whitespace wasn't shown in diff fragments on the review request page.
• Trailing whitespace wasn't always shown in the diff viewer, if it appeared in the middle of a comment block.
• Removed some old YUI files that were still in the repository.

Feature Improvements:

• Improved lighttpd and Apache+fastcgi support. A Review Board install using either lighttpd or Apache+fastcgi should now work out of the box without setting up any custom scripts for launching fastcgi.
• The diff fragments placeholders in the review request page are now set to the same size as the fragment itself. This means that progressively loaded fragments should no longer cause the page to jump nearly as much, resulting in a smoother loading experience.
• When displaying the Review form, we now make sure that the form is scrolled to the top. This fixes a bug where the page was often scrolled to the bottom of the list of comments.
• Several improvements were made to Bazaar support. We now use bzrlib and support revid information in the diff header.
• The comment dialog is now slightly wider and has slightly smaller text, resulting in more room for comments.

New Features:

• Load times for nearly all parts of Review Board have greatly increased. The browser is now instructed to aggressively cache media files and many pages, meaning fewer downloads when going to any particular page.

  We optimize the order in which resources are loaded in order to take advantage of the browser's ability to parallelize downloads of similar resources types (such as CSS files or JavaScript).

  The results of these files are GZip'd to further increase transfer speeds.

  Apache users will get these improvements for free. lighttpd users will need to upgrade their lighttpd config file based on the template provided with Review Board in the contrib/conf/ directory.

  On top of this, we now use jQuery 1.3.1, which is many times faster than the previous version of jQuery we were using, and we also have a number of improvements such as progressive loading of the diff viewer.

• Added progressive loading of the diff viewer. This is one of the main features of this release. Regardless of the size of the diff, the diff viewer now loads quickly, and loads each file's diff progressively. This gives people the ability to review code before all the diffs have even been generated.

  This also fixes some major scalability issues we've had with large diffs. In theory, it's now possible to review very large diffs that span many pages.

• Added support for linkifying pieces of text in the Description and Testing Done fields. (Issue #258)

  The following formats are supported:

  • URLs
  • /r/123 numbers (even /r/123/diff and such)
  • bug/issue numbers (in the form of "bug abc123", "bug #abc123", "issue abc123" and "issue #abc123")
• Added support for Clearcase repositories. This depends on a mounted view on the Review Board server. Patch by Daniel Belz.

After a long time in development, we're finally approaching our landmark 1.0 release. Today we've released Review Board 1.0 alpha 1, which can be downloaded by following the instructions on our downloads page or through our Getting Started guide.

This release has been considered stable enough for production use. A number of companies have been using the nightlies leading up to this release for quite a while. If you've been using nightly releases until now, you'll automatically be switched over to the 1.0 alpha 1 release the next time you upgrade. If you want to keep using nightlies to help test the latest and greatest, see the "Nightly Releases" section in our Getting Started guide for instructions on switching back.

In the following months, we expect to release one more alpha, two betas, a release candidate, and the final 1.0 release. The plans leading up to the 1.0 release are available in our roadmap. We're going to be focusing on complete installation, administration and user documentation, as well as enhancing usability and stabilizing the product. This also means we're going to begin entering a feature freeze, where we'll only accept new features if we agree they're critical to this release. Most new features will be saved until after 1.0.

We'd like to thank the 100+ contributors who have helped make Review Board such an awesome code review tool. If you have questions, need support, or would like to help out with the project, you can contact us on our mailing list or file bug reports. And if you're a happy user, we'd love to hear from you too!