Re: new commitfest transition guidance - Mailing list pgsql-hackers

From Jelte Fennema-Nio
Subject Re: new commitfest transition guidance
Date
Msg-id CAGECzQRy_19u709T6FzTKWDUqgi7vWHMO1nPaijucaORT-R6Xw@mail.gmail.com
Whole thread Raw
In response to Re: new commitfest transition guidance  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: new commitfest transition guidance
List pgsql-hackers
On Thu, 6 Feb 2025 at 01:29, Tom Lane <tgl@sss.pgh.pa.us> wrote:
>
> Peter Geoghegan <pg@bowt.ie> writes:
> > Evidently this new policy is why my skip scan patch series wasn't
> > being tested by CI.
>
> Well no, the reason CI wasn't testing anything was the cfbot was
> broken.  See nearby "CFBot is broken" thread.

That's not entirely true. CFBot never runs on patches that are only in
closed commitfests. So even if it had been working fine, it would have
not picked up Peter G's skip scan patch until after he moved the patch
to the new commitfest.

I definitely agree with Peter G that that's a problem i.e. having
CFBot not run on patches for ~X days until the author notices they
were not moved and moves them. Ofcourse the CFBot could be changed to
behave differently, but then the question becomes how should it behave
then? When do we want to stop running CFBot on patches?

Related: What do we do in general with patches that have been moved?
And when do we do that?



pgsql-hackers by date:

Previous
From: Zhang Mingli
Date:
Subject: Re: Proposal to CREATE FOREIGN TABLE LIKE
Next
From: vignesh C
Date:
Subject: Re: Virtual generated columns