On Fri, Jan 15, 2021 at 4:54 PM japin <japi...@hotmail.com> wrote: > On Fri, 15 Jan 2021 at 18:57, Bharath Rupireddy > <bharath.rupireddyforpostg...@gmail.com> wrote: > > On Fri, Jan 15, 2021 at 4:03 PM Amit Kapila <amit.kapil...@gmail.com> wrote: > >> That sounds like a better way to fix and in fact, I was about to > >> suggest the same after reading your previous email. I'll think more on > >> this but in the meantime, can you add the test case in the patch as > >> requested earlier as well. > > > > @Li Japin Please let me know if you have already started to work on > > the test case, otherwise I can make a 0002 patch for the test case and > > post. > > > > Yeah, I'm working on the test case. Since I'm not familair with the logical > replication test, it may take some times.
Thanks a lot. I quickly added the initial use case where we saw the bug, attached is 0002 patch. Please have a look and add further use cases if necessary. If okay, it's better to make a cf entry. I have one comment in v3-0001 patch, + * There might some relations dropped from the publication, we do I think we should change it to - "There might be some relations". With Regards, Bharath Rupireddy. EnterpriseDB: http://www.enterprisedb.com
v3-0002-Test-behaviour-of-ALTER-PUBLICATION-.-DROP-TABLE.patch
Description: Binary data