On Wed, Jun 15, 2022 at 5:05 PM shiy.f...@fujitsu.com
<shiy.f...@fujitsu.com> wrote:
>
...
> Thanks for updating the patch. Two comments:
>
> 1.
> +     it means the copied table <literal>t3</literal> contains all rows even 
> when
> +     they do not patch the row filter of publication 
> <literal>pub3b</literal>.
>
> Typo. I think "they do not patch the row filter" should be "they do not match
> the row filter", right?
>
> 2.
> @@ -500,7 +704,6 @@
>        </para>
>       </listitem>
>      </itemizedlist></para>
> -
>    </sect2>
>
>    <sect2 id="logical-replication-row-filter-examples">
>
> It seems we should remove this change.
>

Thank you for your review comments. Those reported mistakes are fixed
in the attached patch v3.

------
Kind Regards,
Peter Smith.
Fujitsu Australia

Attachment: v3-0001-PGDOCS-tablesync-ignores-publish-operation.patch
Description: Binary data

Reply via email to