This both simplifies the codebase by reducing duplication, and also
fixes a bug where the committer info was never loaded from the database
rows because the load_* methods had never been updated with the
appropriate row/property setters.