Re: [HACKERS] logical decoding of two-phase transactions

On Thu, Nov 19, 2020 at 11:27 AM Ajin Cherian <itsajin(at)gmail(dot)com> wrote:
>
> > Why did you make a change in stream_start API? I think it should be
> > *_change and *_truncate APIs because the concurrent abort can happen
> > while decoding any intermediate change. If you agree then you can
> > probably take that code into a separate function and call it from the
> > respective APIs.
> >
> Patch 0001:
> Updated this from stream_start to stream_change. I haven't updated
> *_truncate as the test case written for this does not include a
> truncate.
>

I think the same check should be there in truncate as well to make the
APIs consistent and also one can use it for writing another test that
has a truncate operation.

--
With Regards,
Amit Kapila.