RE: Commit Fest 2019-01 is now closed

From: "Tsunakawa, Takayuki" <tsunakawa(dot)takay(at)jp(dot)fujitsu(dot)com>
To: 'Chris Travers' <chris(dot)travers(at)adjust(dot)com>
Cc: Bruce Momjian <bruce(at)momjian(dot)us>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, "Magnus Hagander (magnus(at)hagander(dot)net)" <magnus(at)hagander(dot)net>, Michael Paquier <michael(at)paquier(dot)xyz>, Postgres hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: RE: Commit Fest 2019-01 is now closed
Date: 2019-02-12 07:51:41
Message-ID: 0A3221C70F24FB45833433255569204D1FB977C8@G01JPEXMBYT05
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

From: Chris Travers [mailto:chris(dot)travers(at)adjust(dot)com]
> On Fri, Feb 8, 2019 at 9:40 AM Tsunakawa, Takayuki
> <tsunakawa(dot)takay(at)jp(dot)fujitsu(dot)com> wrote:
> I'm sorry if I repeat what someone proposed in the past, but it
> would be nice if the CF app could show the number of modified lines (added
> + deleted) for each patch, e.g., the last line of diffstat command output.
> That would help young newbies to choose patches.
>
>
>
> An important prerequisite here would be to list all patches on the most
> recent email. Right now it just lists one so you have to follow the link
> to the hackers list entry and look at each patch one at a time.
>
> So if we could grab all attachments ending in .diff or .patch and list line
> counts, that would be a welcome improvement.

I understood it may be difficult to extract all patch files in a single email. Sorry, I can't tell whether it's possible...

Another idea is to add the number of lines of the mail thread. Reviewers can expect that a patch with short discussion may be easy to work on, although it's not always true of course. Long mail threads are hard to understand for those who are new and/or not good at English.

Regards
Takayuki Tsunakawa

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Pavel Stehule 2019-02-12 08:15:34 Re: proposal: variadic argument support for least, greatest function
Previous Message Takashi Menjo 2019-02-12 07:06:53 Re: [HACKERS][PATCH] Applying PMDK to WAL operations for persistent memory