On Friday, April 23, 2021 6:03 PM I wrote:
> I've combined v5 with above accepted comments.
> 
> Just in case, I've conducted make check-world, the test of
> clobber_cache_always mode again for v6 and tight loop test of 100 times for
> 010_truncate.pl.
> The result is that all passed with no failure.
I'm sorry, I realized another minor thing which should be fixed.

In v6, I did below.
+Bitmapset *
+RelationGetIdentityKeyBitmap(Relation relation)
+{
+       Bitmapset  *idindexattrs;       /* columns in the replica identity */
...
+       /* Build attributes to idindexattrs */
+       idindexattrs = NULL;

But, we removed the loop, so we can insert NULL
at the beginning to declare idindexattrs.
v7 is the version to update this part and
related comments from v6.

Best Regards,
        Takamichi Osumi

Attachment: truncate_in_synchronous_logical_replication_v07.patch
Description: truncate_in_synchronous_logical_replication_v07.patch

Reply via email to