On Thu, Dec 2, 2021 at 8:42 AM Amit Langote <amitlangot...@gmail.com> wrote: > > On Thu, Dec 2, 2021 at 9:44 houzj.f...@fujitsu.com > <houzj.f...@fujitsu.com> wrote: > > On Wed, Dec 1, 2021 3:01 PM Amit Kapila <amit.kapil...@gmail.com> wrote: > > > > > On Mon, Nov 22, 2021 at 12:55 PM Amit Langote > > > > > <amitlangot...@gmail.com> wrote: > > > > > > On second thought, I agree that de-duplicating partitions from > > > > > > this view is an improvement. > > > > > > > > > > Fair enough. Hou-San, Can you please submit the updated patch after > > > > > fixing any pending comments including the test case? > > > > > > > > Attach the updated patch which address the comments so far. > > > > > > > > The patch only adds a testcase in publication.sql because the > > > > duplicate output doesn't cause unexpected behavior in pub-sub test. > > > > > > Thanks, the patch looks good to me. I have slightly changed the commit > > > message in the attached. I would like to commit this only in HEAD as > > > there is no > > > user complaint about this and it might not stop any user's service unless > > > it relies > > > on this view's output for the initial table synchronization. > > The patch looks good to me too in that it gets the job done. >
Pushed. -- With Regards, Amit Kapila.
