Dear Amit, hackers, > The patch looks mostly good to me. However, I have one > question/comment as follows: > > - <varlistentry> > + <varlistentry id="sql-createsubscription-binary" xreflabel="binary"> > <term><literal>binary</literal> (<type>boolean</type>)</term> > <listitem> > > To allow references to the binary option, we add the varlistentry id > here. It looks slightly odd to me to add id for just one entry, see > commit 78ee60ed84bb3a1cf0b6bd9a715dcbcf252a90f5 where we have > purposefully added ids to allow future references. Shall we add id to > other options as well on this page?
I have analyzed same points and made patch that could be applied atop v19-0001. Please check 0002 patch. Best Regards, Hayato Kuroda FUJITSU LIMITED
v19-0001-Allow-logical-replication-to-copy-table-in-binary.patch
Description: v19-0001-Allow-logical-replication-to-copy-table-in-binary.patch
v19-0002-Add-XML-ID-attributes-to-create_subscription.sgm.patch
Description: v19-0002-Add-XML-ID-attributes-to-create_subscription.sgm.patch