Re: Adding a note to protocol.sgml regarding CopyData

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

In response to

Responses

Browse pgsql-hackers by date

  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