Re: next CommitFest

From: Robert Haas <robertmhaas(at)gmail(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Brendan Jurd <direvus(at)gmail(dot)com>, David Fetter <david(at)fetter(dot)org>, Chris Browne <cbbrowne(at)acm(dot)org>, pgsql-hackers(at)postgresql(dot)org
Subject: Re: next CommitFest
Date: 2009-11-17 02:14:40
Message-ID: 603c8f070911161814n334de2f2y67f299f71adeb33d@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Mon, Nov 16, 2009 at 6:56 PM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> Brendan Jurd <direvus(at)gmail(dot)com> writes:
>> One of the rewards for getting a patch into the tree is having your
>> name immortalised in the commit log.  There's no such compensation for
>> reviewing patches.
>
> Well, that could be fixed: instead of
>
>        blah blah blah
>
>        Joe Coder
>
> we could write
>
>        blah blah blah
>
>        Joe Coder, reviewed by X and Y
>
> Although keeping track of just who to credit might be a bit tricky.
> I'd be happy to commit to crediting whoever is listed in the CF entry
> for the patch, but sometimes other people have chimed in as much or
> more as the nominal reviewer.

If looking at the CF entries, it's important to note that sometimes
one person posts a review and someone else (historically, me) adds a
link to it, and of course we want to post the reviewer, not the person
who dropped in the link. I try to always make the comment something
like "review from so-and-so" when I do this, but I (or someone) might
forget that on occasion, so clicking through to the underlying message
is probably a good idea.

As for other people chiming in, I think half a loaf is better than
none. We should try to credit the people who deserve credit; and if
someone who chimes in is particularly concerned about getting credit,
then they can post a link to their chiming-in on the CF app.
Otherwise, it's best effort, just like anything else.

...Robert

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Greg Sabino Mullane 2009-11-17 02:23:35 Raising the geqo_threshold default
Previous Message Robert Haas 2009-11-17 02:06:06 Re: next CommitFest