Re: Opinion poll: Sending an automated email to a thread when it gets added to the commitfest - Mailing list pgsql-hackers

From David Rowley
Subject Re: Opinion poll: Sending an automated email to a thread when it gets added to the commitfest
Date
Msg-id CAApHDvqNBMC1zz9JSGxNe-UvBHDOK2dDNvP9MHmjn6ADnJRUDA@mail.gmail.com
Whole thread Raw
In response to Re: Opinion poll: Sending an automated email to a thread when it gets added to the commitfest  (Jelte Fennema-Nio <postgres@jeltef.nl>)
Responses Re: Opinion poll: Sending an automated email to a thread when it gets added to the commitfest
List pgsql-hackers
On Thu, 15 Aug 2024 at 20:00, Jelte Fennema-Nio <postgres@jeltef.nl> wrote:
>
> On Thu, 15 Aug 2024 at 05:17, Euler Taveira <euler@eulerto.com> wrote:
> > If I understand your proposal correctly, there will be another email to the
> > thread if the previous CF was closed and someone opened a new CF entry.
> > Sometimes some CF entries are about the same thread.
>
> Yeah, that's correct. If a new CF entry is created for an existing
> thread a new email would be sent. But to be clear, if CF entry is
> pushed to the next commitfest, **no** new email would be sent.

I was thinking about this today when looking at [1], you can see at
the end of that email someone posted a link to the CF entry.
Unfortunately, that was for the Jan-2024 CF, which is not really that
useful to look at today.

It looks like only about 35% of patches in this CF have "num CFs" = 1,
so it might be annoying to be taken to an old entry 65% of the time
you click the proposed URL. Maybe instead of the URLs showing the CF
number that the patch was first added to, if it could have a reference
in the URL that looks up the maximum CF which contains the patch and
show that one. e.g. https://commitfest.postgresql.org/latest/4751/ .
Without that, if you try to modify the status of a patch in an older
CF, you'll just be told that you can't do that.  (it is true you can
click the latest CF in the status column, but that's at least one more
click than I'd have hoped)

David

[1] https://postgr.es/m/CACJufxEdavJhkUDhJ1jraXnZ9ayNQU+TvjuQjzQbuGS06oNZEQ@mail.gmail.com



pgsql-hackers by date:

Previous
From: Tatsuo Ishii
Date:
Subject: Re: Add memory/disk usage for WindowAgg nodes in EXPLAIN
Next
From: "David G. Johnston"
Date:
Subject: Re: Add RESPECT/IGNORE NULLS and FROM FIRST/LAST options