Dear Shlok,
> Thanks for reviewing the patch. I have updated the patch and attached
> the v2 patch.
Thanks for the update. According to others, different kinds of statements
should be
different else-if statements. So we may have to be something like below:
```
else if (stmt->kind == ALTER_SUBSCRIPTION_REFRESH_PUBLICATION)
{
...
}
else if (stmt->kind == ALTER_SUBSCRIPTION_ADD_PUBLICATION ||
stmt->kind == ALTER_SUBSCRIPTION_DROP_PUBLICATION ||
stmt->kind == ALTER_SUBSCRIPTION_SET_PUBLICATION)
{
if (IsSet(opts.specified_opts, SUBOPT_REFRESH) && opts.refresh)
...
}
```
Best regards,
Hayato Kuroda
FUJITSU LIMITED
