RE: [BUG] wrong refresh when ALTER SUBSCRIPTION ADD/DROP PUBLICATION

[houzj.f...@fujitsu.com]

> -----Original Message-----
> From: Masahiko Sawada <sawada.m...@gmail.com>
> Sent: Tuesday, August 24, 2021 8:52 AM
> 
> Thanks. The patch for HEAD looks good to me. Regarding the patch for PG14, I
> think we need to update the comment as well:
> 
> @@ -987,7 +986,7 @@ AlterSubscription(AlterSubscriptionStmt *stmt, bool
> isTopLevel)
>       PreventInTransactionBlock(isTopLevel, "ALTER SUBSCRIPTION with
> refresh");
> 
>       /* Only refresh the added/dropped list of publications. */
> -     sub->publications = stmt->publication;
> +     sub->publications = publist;
> 

Thanks for the comment, attach new version patch which fixed it.

Best regards,
Hou zj

v8-0001-PG15-Fix-Alter-Subscription-Add-Drop-Publication-refre.patch
Description: v8-0001-PG15-Fix-Alter-Subscription-Add-Drop-Publication-refre.patch

v8-0001-PG14-Fix-Alter-Subscription-Add-Drop-Publication-refre_patch
Description: v8-0001-PG14-Fix-Alter-Subscription-Add-Drop-Publication-refre_patch