Re: Releasing in September

From: Noah Misch <noah(at)leadboat(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Magnus Hagander <magnus(at)hagander(dot)net>, Andres Freund <andres(at)anarazel(dot)de>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>, Bruce Momjian <bruce(at)momjian(dot)us>, Robert Haas <robertmhaas(at)gmail(dot)com>
Subject: Re: Releasing in September
Date: 2016-01-22 04:56:26
Message-ID: 20160122045626.GA3665868@tornado.leadboat.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Wed, Jan 20, 2016 at 12:40:04PM -0500, Tom Lane wrote:
> I think it might also be a good idea if we could somehow distinguish
> "nobody had time for that yet" from "nobody is interested", with an eye
> to flat-out rejecting patches that no one cares enough about to review.
> Maybe we could reduce the workload by inventing some kind of up-front
> filter whereby a patch isn't even a candidate to get reviewed unless, say,
> at least two people besides the author say "this sounds like it's worth
> pursuing".
>
> One of the other things I do not like about the current CF process is that
> it's created a default assumption that most submitted patches should get
> accepted eventually. It is *very* hard to reject a patch altogether in
> the CF process: you more or less have to show cause why it should not get
> accepted. That default is backwards. Maybe this isn't the process' fault
> exactly, but it sure seems like that's how we approach patches now.

These paragraphs point to different facets of the same problem: community
conventions make rejecting a patch a substantial project in its own right. +1
for experimenting with one or more ways to expedite that. There's your
two-ACKs idea above, and we floated[1] a few more at the 2015-06-16 meeting:

This was followed by discussion of how we arrange the CFs. Noah suggested a
prioritization system. He suggested a system where various committers can
provide feedback on stuff as triage. Simon agreed and volunteered. Haas
suggested a 2-committer veto. Frost said we need a formal way to "nack"
something. We need to triage stuff earlier in the process. Haas says the big
problem is the endless arguments because we don't want to say "no" to people
because we have scarce committer time.

I recommend trying the two-ACKs plan for one CF. Keep it if it helps
markedly; otherwise, try another of these variants for the CF after that one.

Thanks,
nm

[1] https://wiki.postgresql.org/wiki/PgCon_2015_Developer_Meeting#9.6_Schedule

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Noah Misch 2016-01-22 05:07:22 Re: Releasing in September
Previous Message Amit Langote 2016-01-22 04:41:54 Re: Declarative partitioning