On Fri, Jan 6, 2023 at 11:18 AM Dilip Kumar <dilipbal...@gmail.com> wrote: > > > > > To fix it, One idea is to send a stream abort message when we are cleaning > > up > > crashed transaction on publisher(e.g. in ReorderBufferAbortOld()). And here > > is > > a tiny patch which changes the same. I have confirmed that the bug is fixed > > and > > all regression tests pass. I didn't add a testcase because we need to make > > sure > > the crash happens before all the WAL logged transactions data are decoded > > which > > doesn't seem easy to write a stable test for this. > > > > Thoughts ? > > Fix looks good to me. Thanks for working on this. >
Pushed! -- With Regards, Amit Kapila.
