Patch review process; Warning to patch submitters

Started by Bruce Momjianover 18 years ago3 messages
#1Bruce Momjian
bruce@momjian.us

Based on the patches status page:

http://developer.postgresql.org/index.php/Todo:PatchStatus#current_8.3_patch_status

we are a few weeks away from completing all patch review and entering
beta. If you have been asked to updated your patch by a patch reviewer,
please do so in the next few days or your patch risks being held for
8.4.

Patch reviewers are not going to chase down patch authors to get patch
changes completed; one email request is sufficient, and I will back up
any patch reviewer who wants to hold a patch for 8.4 because the patch
was not updated by the patch author in a timely manner.

This holds true not only for patches where an update has already been
requested but for future requests for updates. The original patch might
have made the April 1 deadline, but if it isn't updated promptly, then
it will be considered to be not ready to apply and be held for 8.4.

The bottom line is that the patch review process is hard enough, and we
need the assistance of patch submitters to work with the patch reviewers
in a timely manner.

--
Bruce Momjian <bruce@momjian.us> http://momjian.us
EnterpriseDB http://www.enterprisedb.com

+ If your life is a hard drive, Christ can be your backup. +

#2Simon Riggs
simon@2ndquadrant.com
In reply to: Bruce Momjian (#1)
Re: Patch review process; Warning to patch submitters

On Fri, 2007-06-08 at 16:29 -0400, Bruce Momjian wrote:

The bottom line is that the patch review process is hard enough, and we
need the assistance of patch submitters to work with the patch reviewers
in a timely manner.

This includes me, so: Understood.

There is a patch missing from the patch status page:
http://archives.postgresql.org/pgsql-hackers/2007-03/msg01795.php
http://archives.postgresql.org/pgsql-patches/2007-04/msg00250.php

Should I add it myself?

--
Simon Riggs
EnterpriseDB http://www.enterprisedb.com

#3Alvaro Herrera
alvherre@commandprompt.com
In reply to: Simon Riggs (#2)
Re: Patch review process; Warning to patch submitters

Simon Riggs wrote:

On Fri, 2007-06-08 at 16:29 -0400, Bruce Momjian wrote:

The bottom line is that the patch review process is hard enough, and we
need the assistance of patch submitters to work with the patch reviewers
in a timely manner.

This includes me, so: Understood.

There is a patch missing from the patch status page:
http://archives.postgresql.org/pgsql-hackers/2007-03/msg01795.php
http://archives.postgresql.org/pgsql-patches/2007-04/msg00250.php

Should I add it myself?

Sure, please do.

--
Alvaro Herrera http://www.amazon.com/gp/registry/5ZYLFMCVHXC
"El sudor es la mejor cura para un pensamiento enfermo" (Bardia)