From: | Simon Riggs <simon(at)2ndquadrant(dot)com> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Bruce Momjian <bruce(at)momjian(dot)us>, Andres Freund <andres(at)anarazel(dot)de>, Pavan Deolasee <pavan(dot)deolasee(at)gmail(dot)com>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: pgsql: New files for MERGE |
Date: | 2018-04-11 20:00:22 |
Message-ID: | CANP8+jLvv_QYRfn1TbBOuFN42JF5CPZP_DPHLzugB_o1-UzKkw@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers pgsql-hackers |
On 7 April 2018 at 18:45, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> Simon Riggs <simon(at)2ndquadrant(dot)com> writes:
>> On 6 April 2018 at 17:22, Bruce Momjian <bruce(at)momjian(dot)us> wrote:
>>> My point was that people didn't ask you to work harder on fixing the
>>> patch, but in reverting it. You can work harder on fixing things in the
>>> hope they change their minds, but again, that isn't addressing their
>>> request.
>
>> If Tom or Andres still feel that their concerns have not been
>> addressed over the last few days, I am happy to revert the patch with
>> no further discussion from me in this cycle.
>
> FWIW, I still vote to revert. Even if the patch were now perfect,
> there is not time for people to satisfy themselves of that, and
> we've got lots of other things on our plates.
>
> I'd be glad to participate in a proper review of this when v12
> opens. But right now it just seems too rushed, and I have little
> confidence in it being right.
>
> regards, tom lane
>
> PS: If you do revert, please wrap it up as a single revert commit,
> not a series of half a dozen. You've already put several
> non-buildable states into the commit history as a result of this
> patch, each one of which is a land mine for git bisect testing.
> We don't need more of those. Also, looking at the reverse of the
> reversion commit will provide a handy way of seeing the starting
> point for future discussion of this patch.
Will do.
"Commence primary ignition."
--
Simon Riggs http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
From | Date | Subject | |
---|---|---|---|
Next Message | Heikki Linnakangas | 2018-04-11 21:01:16 | pgsql: Make local copy of client hostnames in backend status array. |
Previous Message | Alvaro Herrera | 2018-04-11 18:33:09 | pgsql: Fix ALTER TABLE .. ATTACH PARTITION ... DEFAULT |
From | Date | Subject | |
---|---|---|---|
Next Message | Simon Riggs | 2018-04-11 20:12:57 | Re: Bugs in TOAST handling, OID assignment and redo recovery |
Previous Message | Andrew Gierth | 2018-04-11 19:57:12 | Re: lazy detoasting |