RE: Allow logical replication to copy tables in binary format

On Tuesday, March 14, 2023 8:02 PM Melih Mutlu <m.melihmutlu@gmail.com> wrote:
> Attached v13.
Hi,


Thanks for sharing v13. Few minor review comments.
(1) create_subscription.sgml

+          column types as opposed to text format. Even when this option is enabled,
+          only data types having binary send and receive functions will be
+          transferred in binary. Note that the initial synchronization requires

(1-1)

I think it's helpful to add a reference for the description about send and receive functions (e.g. to the page of
CREATETYPE). 

(1-2)

Also, it would be better to have a cross reference from there to this doc as one paragraph probably in "Notes". I
suggestedthis, because send and receive functions are described as "optional" there and missing them leads to error in
thecontext of binary table synchronization. 

(3) copy_table()

+       /*
+        * If the publisher version is earlier than v14, it COPY command doesn't
+        * support the binary option.
+        */

This sentence doesn't look correct grammatically. We can replace "it COPY command" with "subscription" for example.
Kindlyplease fix it. 


Best Regards,
    Takamichi Osumi