Re: Logical Replication and triggers

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

In response to

Responses

Browse pgsql-hackers by date

  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