From: | Simon Riggs <simon(at)2ndquadrant(dot)com> |
---|---|
To: | Thomas Rosenstein <thomas(dot)rosenstein(at)creamfinance(dot)com> |
Cc: | Robert Haas <robertmhaas(at)gmail(dot)com>, Craig Ringer <craig(at)2ndquadrant(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Logical Replication and triggers |
Date: | 2017-11-21 21:28:30 |
Message-ID: | CANP8+jJA4iq=BRdr_Cv5dEAXZ80x8CPz5L3D046Q-CZUG2-N=A@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 21 November 2017 at 16:13, Thomas Rosenstein
<thomas(dot)rosenstein(at)creamfinance(dot)com> wrote:
> To weigh in here, I actually find it's a big hurdle
>
> I'm a postgres user and not a postgres dev, so I definitly have the feeling
> I'm not qualified to answer if this really does what it's intended todo.
> Further more not beeing in the processes it will take me probably 2 - 3
> hours (if I have time) to figure out everything I should do and how I should
> do it,
> somebody doing this regularly might take 5 minutes.
>
> Yes it fixes my replication issues, yes it seems to work on the first look,
> but what does it really do - no idea!
Fair enough, but that's not what Craig asked is it?
Seems like he gave a helpful, long explanation of how you can help
expedite the process of fixing the bug, to which he received no reply.
We're trying to ensure that bugs are fixed, but also take care not to
introduce further bugs or false fixes that don't move us forwards.
I would have acted on this myself a few days back if I thought the
patch was correct, but I see multiple command counter increments
there, so suspect an alternate fix is correct.
--
Simon Riggs http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
From | Date | Subject | |
---|---|---|---|
Next Message | Robert Haas | 2017-11-21 21:31:45 | Re: default range partition and constraint exclusion |
Previous Message | Peter Geoghegan | 2017-11-21 21:19:51 | Re: [HACKERS] CLUSTER command progress monitor |