Re: Commitfest Update

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Robert Haas <robertmhaas(at)gmail(dot)com>
Cc: Pavel Borisov <pashkin(dot)elfe(at)gmail(dot)com>, Matthias van de Meent <boekewurm+postgres(at)gmail(dot)com>, Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org>, Julien Rouhaud <rjuju123(at)gmail(dot)com>, Greg Stark <stark(at)mit(dot)edu>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: Commitfest Update
Date: 2022-03-31 14:37:11
Message-ID: 552155.1648737431@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Robert Haas <robertmhaas(at)gmail(dot)com> writes:
> On Thu, Mar 31, 2022 at 10:11 AM Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>> ... Would it be feasible or reasonable
>> to drop reviewers if they've not commented in the thread in X amount
>> of time?

> In theory, this might cause someone who made a valuable contribution
> to the discussion to not get credited in the commit message. But it
> probably wouldn't in practice, because I at least always construct the
> list of reviewers from the thread, not the CF app, since that tends to
> be wildly inaccurate in both directions. So maybe it's fine? Not sure.

Hmm, I tend to believe what's in the CF app, so maybe I'm dropping the
ball on review credits :-(. But there are various ways we could implement
this. One way would be a nagbot that sends private email along the lines
of "you haven't commented on patch X in Y months. Please remove your name
from the list of reviewers if you don't intend to review it any more."

regards, tom lane

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Devrim Gündüz 2022-03-31 14:38:39 Re: head fails to build on SLES 12
Previous Message Aleksander Alekseev 2022-03-31 14:35:30 Re: XID formatting and SLRU refactorings (was: Add 64-bit XIDs into PostgreSQL 15)