Bruce Momjian <pgman@candle.pha.pa.us> writes:
> Marc G. Fournier wrote:
>> On Sat, 4 Dec 2004, Bruce Momjian wrote:
>>> OK, so what do we want the process to be?
>>
>> Basically, during Beta/Release, we should almost a policy where a third
>> party patch needs to be approved by a second committer *before* being
>> applied ... and that even applies to Tom >:) Your own patch, fine ... but
>> a third party patch, even submitted by someone who has submitted patch
>> previously, should be reviewed/approved by two committers ...
> Please find a cure that isn't worse than the disease. I don't have time
> to apply patches as it is, let alone check with someone else.
That's a fair objection, but if it means that the default is that
patches don't get applied during late beta/RC, I'm not sure I'm unhappy
with that default.
In the particular case of this patch, although Bruce said that others
had already commented on the patch, the only comments I see in the
pgpatches archives said that the patch was unreviewable because it
wasn't offered as a diff. I think it would be reasonable to insist on
at least one concurrence ("looks ok to me") posted to pgsql-patches
before applying during late beta. We've gotten into a mode where
if you like a patch you say nothing, but I wonder whether we shouldn't
change that habit.
regards, tom lane