Re: Online enabling of checksums
Andres Freund <andres@anarazel.de>
From: Andres Freund <andres@anarazel.de>
To: "Joshua D. Drake" <jd@commandprompt.com>
Cc: Magnus Hagander <magnus@hagander.net>, Andrey Borodin <x4mmm@yandex-team.ru>, Tomas Vondra <tomas.vondra@2ndquadrant.com>, Heikki Linnakangas <hlinnaka@iki.fi>, Daniel Gustafsson <daniel@yesql.se>, Michael Banck <michael.banck@credativ.de>, Robert Haas <robertmhaas@gmail.com>, Greg Stark <stark@mit.edu>, PostgreSQL-development <pgsql-hackers@postgresql.org>
Date: 2018-04-05T21:01:33Z
Lists: pgsql-hackers
On 2018-04-05 13:51:41 -0700, Joshua D. Drake wrote: > On 04/05/2018 01:12 PM, Andres Freund wrote: > > I want to be on the record that I think merging a nontrival feature that > > got submitted 2018-02-21, just before the start of the last last CF, is > > an abuse of process, and not cool. We've other people working hard to > > follow the process, and circumventing it like this just signals to > > people trying to follow the rules that they're fools. > > > > Merging ~2kloc patches like that is going to cause pain. And even if > > not, it causes procedual damage. > > Perhaps I am missing something but there has been a lot of public discussion > on this feature for the last 7 weeks of which you barely participated. I've commented weeks ago about my doubts, and Robert concurred: http://archives.postgresql.org/message-id/CA%2BTgmoZPRfMqZoK_Fbo_tD9OH9PdPFcPBsi-sdGZ6Jg8OMM2PA%40mail.gmail.com > I certainly understand wanting some notice before commit but there has > been lots of discussion, multiple people publicly commenting on the > patch and Magnus has been very receptive to all feedback (that I have > seen). It's perfectly reasonable to continue review / improvement cycles of a patch, even if it's not going to get in the current release. What does that have to do with what I am concerned about? > Perhaps we are being a sensitive because of another patch that is > actually ramrodding the process and we need to take a step back? No. See link above. Please don't use "we" in this childishness implying fashion. - Andres
Commits
-
Online enabling and disabling of data checksums
- f19c0eccae96 19 (unreleased) landed
-
Deactive flapping checksum isolation tests.
- bf75fe47e444 11.0 landed
-
Add support for coordinating record typmods among parallel workers.
- cc5f81366c36 11.0 cited