What about a mentoring schema in order to push up the gap that
represents catching up with cases like the one Andrew posted?
By the way, being a patch reviewer doesn't represents also to be able
to find out potential problems in the code, which may have nothing to
do with the patch functionality?
g.-
On 5/16/07, Bruce Momjian <bruce@momjian.us> wrote:
> Andrew Dunstan wrote:
> >
> >
> > Bruce Momjian wrote:
> > > In talking to people who are assigned to review patches or could review
> > > patches, I often get the reply, "Oh, yea, I need to do that".
> > >
> > > Folks, we are six weeks into feature freeze and have made slim progress
> > > on getting patches reviewed and applied. As I stated earlier, we are
> > > now looking at August/September for beta, but that might be pushed back
> > > even later if we don't get more progress.
> > >
> > > It seems there is a lot of reliance on Tom to get the patches applied,
> > > but I don't think that is fair or reasonable. I think we need more
> > > urgency on the part of everyone to make faster progress. Patch
> > > reviewers and committers need to take more initiative to get things done
> > > rather than wait for some external force to prompt them.
> > >
> > >
> >
> > I at least feel uncomfortable about reviewing code that deals with areas
> > I have not touched much, and where I feel the author probably knows a
> > lot more than me. The chance of my catching errors/problems in such a
> > case is much lower.
>
> Yep, that is part of our problem, but even items people have already
> said they _can_ review have shown little progress.
>
> --
> 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. +
>
> ---------------------------(end of broadcast)---------------------------
> TIP 3: Have you checked our extensive FAQ?
>
> http://www.postgresql.org/docs/faq
>
--
Guido Barosio
-----------------------
http://www.globant.com
guido.barosio@globant.com
