From: | Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> |
---|---|
To: | Tatsuo Ishii <ishii(at)sraoss(dot)co(dot)jp> |
Cc: | Fabien COELHO <coelho(at)cri(dot)ensmp(dot)fr>, bpd0018(at)gmail(dot)com, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Adding a note to protocol.sgml regarding CopyData |
Date: | 2018-09-22 11:57:04 |
Message-ID: | CAA4eK1K_YPOb_p43hD6gTyXYSgmEj6GmDVkW0wffeJzYpovUYQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Mon, Sep 10, 2018 at 3:54 PM Tatsuo Ishii <ishii(at)sraoss(dot)co(dot)jp> wrote:
>
> >> Hello Bradley & Tatsuo-san,
> >>
> >>>> ... references to the protocol version lacks homogeneity.
> >>>> ... I'd suggest to keep "the vX.0 protocol" for a short version,
> >>>> and "the version X.0 protocol" for long ...
> >>>
> >>> I agree. Change made.
> >>
> >> Patch applies cleanly. Doc build ok.
> >>
> >> One part talks about "terminating line", the other of "termination
> >> line". I wonder whether one is better than the other?
> >
> > I am not a native English speaker so I maybe wrong... for me, current
> > usage of "terminating line", and "termination line" looks correct. The
> > former refers to concrete example thus uses "the", while the latter
> > refers to more general case thus uses "an".
> >
> > BTW, I think the patch should apply to master and REL_11_STABLE
> > branches at least. But should this be applied to other back branches
> > as well?
>
> I have marked this as "ready for committer".
>
My first thought on this patch is that why do we want to duplicate the
same information in different words at three different places. Why
can't we just extend the current Note where it is currently with some
more information about CopyDone message and then add a reference to
that section in other two places?
--
With Regards,
Amit Kapila.
EnterpriseDB: http://www.enterprisedb.com
From | Date | Subject | |
---|---|---|---|
Next Message | Amit Kapila | 2018-09-22 13:23:23 | Re: New function pg_stat_statements_reset_query() to reset statistics of a specific query |
Previous Message | Pavel Stehule | 2018-09-22 07:02:25 | Re: Proposal for disk quota feature |