On Wed, Feb 3, 2021 at 6:38 AM Peter Smith <smithpb2...@gmail.com> wrote: > > On Wed, Feb 3, 2021 at 12:26 AM Amit Kapila <amit.kapil...@gmail.com> wrote: > > > > On Tue, Feb 2, 2021 at 3:31 PM Peter Smith <smithpb2...@gmail.com> wrote: > > > > > > After seeing Ajin's test [ac0202] which did a DROP TABLE, I have also > > > tried a simple test where I do a DROP TABLE with very bad timing for > > > the tablesync worker. It seems that doing this can cause the sync > > > worker's MyLogicalRepWorker->relid to become invalid. > > > > > > > I think this should be fixed by latest patch because I have disallowed > > drop of a table when its synchronization is in progress. You can check > > once and let me know if the issue still exists? > > > > FYI - I confirmed that the problem scenario that I reported yesterday > is no longer possible because now the V25 patch is disallowing the > DROP TABLE while the tablesync is still running. >
Thanks for the confirmation. BTW, can you please check if we can reproduce that problem without this patch? If so, we might want to apply this fix irrespective of this patch. If not, why not? -- With Regards, Amit Kapila.
