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
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) |