Re: Huge commitfest app update upcoming: Tags, Draft CF, Help page, and automated commitfest creat/open/close - Mailing list pgsql-hackers

From Peter Eisentraut
Subject Re: Huge commitfest app update upcoming: Tags, Draft CF, Help page, and automated commitfest creat/open/close
Date
Msg-id c3d1bd20-5823-44c6-84e0-9a964f703fd9@eisentraut.org
Whole thread Raw
In response to Re: Huge commitfest app update upcoming: Tags, Draft CF, Help page, and automated commitfest creat/open/close  ("David G. Johnston" <david.g.johnston@gmail.com>)
Responses Re: Huge commitfest app update upcoming: Tags, Draft CF, Help page, and automated commitfest creat/open/close
List pgsql-hackers
On 20.06.25 16:41, David G. Johnston wrote:
>     I sense there could be some confusion whether such draft patches
>     should go into the regular commit fest or the draft commit fest, and
>     then also when they should move between them.
> 
> Draft CF patches with “Needs Review” are looking for feedback from 
> others or otherwise aid in development for a patch that isn’t ready to 
> be committed even if said review turns up nothing or is otherwise fully 
> resolved.  Patches in Drafts are never marked Ready to Commit, they get 
> moved to Open first.
> 
> It will be nice if people spend time providing reviews/feedback to 
> patches in Drafts when requested.
> 
> It’s purely the author’s judgement on the readiness of the patch, 
> whether absent our policy they would mark it ready to commit or not.  If 
> they believe it is it should be moved to open, if no, it should remain 
> in drafts.  This is mostly like what happens today but with a clear 
> delineation between reviews to help and reviews to approve commit-ability.
> 
> Otherwise, it’s a place where author patches can sit without having to 
> be bumped to the next cf every other month and where an author patch can 
> be ignored by everyone else not authoring it.

I don't know about this.  This could become an ongoing source of 
confusion, without any clear benefit.  Either the draft and the "real" 
commitfest are going to be indistinguishable, because they are just two 
places to look for stuff to review in various phases of maturity.  Or 
the draft commitfest is just not going to get any attention, which will 
be annoying for those who put things there hoping to get attention.




pgsql-hackers by date:

Previous
From: Peter Eisentraut
Date:
Subject: Re: pg_recvlogical cannot create slots with failover=true
Next
From: "David G. Johnston"
Date:
Subject: Re: Huge commitfest app update upcoming: Tags, Draft CF, Help page, and automated commitfest creat/open/close