On 12.08.21 04:52, Masahiko Sawada wrote:
On Wed, Aug 11, 2021 at 5:42 PM Daniel Gustafsson <dan...@yesql.se> wrote:

On 11 Aug 2021, at 09:57, Masahiko Sawada <sawada.m...@gmail.com> wrote:

Additionally, refresh options as described in
<replaceable>refresh_option</replaceable> of
<literal>REFRESH PUBLICATION</literal> may be specified,
except in the case of <literal>DROP PUBLICATION</literal>.

Since this paragraph is under the literal option “refresh”, which takes a
value, I still find your original patch to be the clearest.

Yeah, I prefer my original patch over this idea. On the other hand, I
can see the point of review comment on it that Amit pointed out[1].

How about this:

-      Additionally, refresh options as described
-      under <literal>REFRESH PUBLICATION</literal> may be specified.
+      Additionally, the options described under <literal>REFRESH
+ PUBLICATION</literal> may be specified, to control the implicit refresh
+      operation.


Reply via email to