Dear Amit, > Thanks, the patch looks mostly good to me. I have made few cosmetic > changes in the attached and combined both the patches.
Thanks, it looks good to me. > The existing > Alter Publication ... Rename tests don't test invalidations arising > from that command. As this patch changes that code path, it would be > good to add a few tests for the same. We can add one for individual > relations and another for ALL Tables publication. I created new patch which adds a test code. I added in 007_ddl.pl, but I feel it is OK to introduce new test file. How do you think? Best regards, Hayato Kuroda FUJITSU LIMITED
v12-0001-Avoid-invalidating-all-RelationSyncCache-entries.patch
Description: v12-0001-Avoid-invalidating-all-RelationSyncCache-entries.patch
v12-0002-Add-testcase.patch
Description: v12-0002-Add-testcase.patch
