From: | Vladimir Gordiychuk <folyga(at)gmail(dot)com> |
---|---|
To: | Craig Ringer <craig(at)2ndquadrant(dot)com> |
Cc: | PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>, Álvaro Herrera <alvherre(at)2ndquadrant(dot)com> |
Subject: | Re: Stopping logical replication protocol |
Date: | 2016-05-10 22:47:11 |
Message-ID: | CAFgjRd0Q98s9_TfDvhPNxSQfiP7RpMWU=Ob7D9eenuDaY+T=_Q@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
>
> Same thread, I just think these are two somewhat separate changes. One is
> just in the walsender and allows return to command mode during waiting for
> WAL. The other is more intrusive into the reorder buffer etc and allows
> aborting decoding during commit processing. So two separate patches make
> sense here IMO, one on top of the other.
About the second part of the patch. What the reason decode and send whole
transaction? Why we can't process logical decoding via WalSndLoop LSN by
LSN as it work in physycal replication? For example if transaction contains
in more them one LSN, first we decode and send "begin", "part data from
current LSN" and then returns to WalSndLoop on the next iteration we send
"another part data", "commit". I don't research in this way, because I
think it will be big changes in comparison callback that stop sending.
2016-05-10 20:22 GMT+03:00 Craig Ringer <craig(at)2ndquadrant(dot)com>:
> On 11 May 2016 at 01:15, Vladimir Gordiychuk <folyga(at)gmail(dot)com> wrote:
>
>> in which release can be include first part?
>>
>
> Since it's not a bug fix, I don't think it can go in before 9.7.
>
>
> --
> Craig Ringer http://www.2ndQuadrant.com/
> PostgreSQL Development, 24x7 Support, Training & Services
>
From | Date | Subject | |
---|---|---|---|
Next Message | Jeff Janes | 2016-05-10 22:53:38 | Re: HeapTupleSatisfiesToast() busted? (was atomic pin/unpin causing errors) |
Previous Message | Andres Freund | 2016-05-10 22:44:25 | Re: HeapTupleSatisfiesToast() busted? (was atomic pin/unpin causing errors) |