Re: Adding a note to protocol.sgml regarding CopyData

2018-10-01 Thread Michael Paquier
On Fri, Sep 28, 2018 at 08:16:46PM +0200, Peter Eisentraut wrote: > Yeah, I don't see why we need to document it three times in the same > chapter. > > Also, that chapter is specifically about version 3.0 of the protocol, so > documenting version 2.0 is out of scope. This has been marked as

Re: Adding a note to protocol.sgml regarding CopyData

2018-09-28 Thread Peter Eisentraut
On 25/09/2018 13:55, Amit Kapila wrote: > On Tue, Sep 25, 2018 at 4:51 AM Bradley DeJong wrote: >> >> On 2018-09-22, Amit Kapila wrote ... >> > ... duplicate the same information in different words at three >> different places ... >> >> I count 7 different places. In the protocol docs, there is

Re: Re[2]: Adding a note to protocol.sgml regarding CopyData

2018-09-25 Thread Amit Kapila
On Tue, Sep 25, 2018 at 4:51 AM Bradley DeJong wrote: > > On 2018-09-22, Amit Kapila wrote ... > > ... duplicate the same information in different words at three > different places ... > > I count 7 different places. In the protocol docs, there is the old > mention in the "Summary of Changes

Re[2]: Adding a note to protocol.sgml regarding CopyData

2018-09-24 Thread Bradley DeJong
Thanks for the feedback. On 2018-09-22, Amit Kapila wrote ... > ... Why can't we just extend the current Note where it is currently ... Because information about how the protocol works belongs in the protocol documentation not in the documentation for one implementation of the protocol.

Re: Adding a note to protocol.sgml regarding CopyData

2018-09-22 Thread Amit Kapila
On Mon, Sep 10, 2018 at 3:54 PM Tatsuo Ishii 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

Re: Adding a note to protocol.sgml regarding CopyData

2018-09-10 Thread Tatsuo Ishii
>> 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. >> >>

Re: Adding a note to protocol.sgml regarding CopyData

2018-09-02 Thread Tatsuo Ishii
> 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

Re[3]: Adding a note to protocol.sgml regarding CopyData

2018-09-02 Thread Fabien COELHO
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

Re[3]: Adding a note to protocol.sgml regarding CopyData

2018-08-27 Thread Bradley DeJong
On 2018-08-27, Fabien COELHO wrote ... > ... 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. protocol.v3.patch Description: Binary data

Re: Adding a note to protocol.sgml regarding CopyData

2018-08-27 Thread Tatsuo Ishii
> Hello Bradley & Tatsuo-san, > > My 0.02€ on the text: > >> Version 2.0 of the PostgreSQL protocol >> In the v3.0 protocol, >> the 3.0 protocol >> version 3.0 of the copy-in/copy-out sub-protocol >> the V2.0 protocol. > > While reading nice English (I learned "holdover"), it occurs to me >

Re[2]: Adding a note to protocol.sgml regarding CopyData

2018-08-27 Thread Fabien COELHO
Hello Bradley & Tatsuo-san, My 0.02€ on the text: Version 2.0 of the PostgreSQL protocol In the v3.0 protocol, the 3.0 protocol version 3.0 of the copy-in/copy-out sub-protocol the V2.0 protocol. While reading nice English (I learned "holdover"), it occurs to me that references to the

Re: Adding a note to protocol.sgml regarding CopyData

2018-08-26 Thread Tatsuo Ishii
> On 2018-08-25, Tatsuo Ishii wrote to the pgsql-docs mailing list ... >> Hi Bradley, >> Thank you for your follow up. Your patch looks good to me. >> Can you please re-send your message in pgsql-hackers attaching to this >> thread ... >> CommitFest app does not allow ... emails other than posted

Re[2]: Adding a note to protocol.sgml regarding CopyData

2018-08-26 Thread Bradley DeJong
On 2018-08-25, Tatsuo Ishii wrote to the pgsql-docs mailing list ... > Hi Bradley, > Thank you for your follow up. Your patch looks good to me. > Can you please re-send your message in pgsql-hackers attaching to this thread ... > CommitFest app does not allow ... emails other than posted to

Re: Adding a note to protocol.sgml regarding CopyData

2018-07-31 Thread Fabien COELHO
Hello Tatsuo-san, Minor suggestions, although I'm not a native English speaker. Well, I did not intend to enhance libpq.sgml but maybe your points is valid (I cannot judge because I am not an native English speaker). Argh, sorry, I did not read the right part:-( The note looks good to

Re: Adding a note to protocol.sgml regarding CopyData

2018-07-31 Thread Tatsuo Ishii
Hi Fabien, Thank you for the comment. > Hello Tatsuo-san, > > Minor suggestions, although I'm not a native English speaker. > >> In libpq.sgml following is stated: >> >>Before PostgreSQL protocol 3.0, it was >>necessary >>for the application to explicitly send the two

Re: Adding a note to protocol.sgml regarding CopyData

2018-07-30 Thread Fabien COELHO
Hello Tatsuo-san, Minor suggestions, although I'm not a native English speaker. In libpq.sgml following is stated: Before PostgreSQL protocol 3.0, it was necessary for the application to explicitly send the two characters \. as a final line to indicate to the server

Adding a note to protocol.sgml regarding CopyData

2018-07-29 Thread Tatsuo Ishii
In libpq.sgml following is stated: Before PostgreSQL protocol 3.0, it was necessary for the application to explicitly send the two characters \. as a final line to indicate to the server that it had finished sending COPY data. While this still works, it is