News

Welcome to End Point’s blog

Ongoing observations by End Point people

PostgreSQL switches to Git

Looks like the Postgres project is finally going to be bite the bullet and switch to git as the canonical VCS. Some details are yet to be hashed out, but the decision has been made and a new repo will be built soon. Now to lobby to get that commit-with-inline-patches list to be created...

7 comments:

David Christensen said...

So are they rebuilding the master repository, compared to the version they've had on git.postgresql.org, or keeping the incremental CVS import until switchover so $hackers don't have to be sad their SHA1s are all changing?

Greg Sabino Mullane said...

The hashes on git.postgresql.org will change. No way around that, but it's a very useful one-time pain!

Jon Jensen said...

The mailing list for commits with inline diffs would be easy enough to create on our own. I'm not sure what good commit notifications without diffs are, really. They either mean I have to manually go look up the diff, or I just don't and then there's less code review going on.

Greg Smith said...

At a minimum we can provide a gitweb URL, so that it's as simple as clicking on the link in the e-mail if you actually want to review the code. That doesn't risk sending out giant diffs and correspondingly bloating the mailing list, while still making it easy to see them if you want to.

@David: the swap from the current incrementally updated repo to a better quality one-time conversion is only going to happen at the switchover. So there will be a single bit of disruption that changes all the commit history, but you'll know it's coming long in advance, and the incremental one will be available until then.

Jon Jensen said...

Greg, yes, the URLs are nice, but in my experience code review happens much more when the diff is right there inline (not in an attachment, right inline). Even "huge" diffs are rarely all that huge, especially compared to some of the bloated HTML email kicking around the net these days. :)

In any case, it doesn't need to be either/or. It'd be easy to have one list that gets commit info + link only, and another that also gets inline diffs, so people can choose which they prefer.

The move to Git is great news, by the way!

Nicolas Marchildon said...

Anyone used Gerrit for code reviews?

Greg Sabino Mullane said...

Nicolas: Not in the Postgres community it seems. I've never heard anyone talk about it, and a mailing list search (http://postgresql.markmail.org/search/?q=gerrit) finds no relevant hits.