Re: Aw: Re: Minor documentation error regarding streaming replication protocol

2020-11-12 Thread Bruce Momjian
On Thu, Oct 15, 2020 at 12:43:22PM -0400, Bruce Momjian wrote: > On Thu, Oct 15, 2020 at 12:01:21PM -0400, Tom Lane wrote: > > Bruce Momjian writes: > > > We would want the timeline history file contents label changed from > > > BYTEA to TEXT in the docs changed for all supported versions, add a C

Re: Aw: Re: Minor documentation error regarding streaming replication protocol

2020-10-15 Thread Bruce Momjian
On Thu, Oct 15, 2020 at 12:01:21PM -0400, Tom Lane wrote: > Bruce Momjian writes: > > We would want the timeline history file contents label changed from > > BYTEA to TEXT in the docs changed for all supported versions, add a C > > comment to all backbranches that BYTEA is the same as TEXT protoco

Re: Aw: Re: Minor documentation error regarding streaming replication protocol

2020-10-15 Thread Tom Lane
Bruce Momjian writes: > We would want the timeline history file contents label changed from > BYTEA to TEXT in the docs changed for all supported versions, add a C > comment to all backbranches that BYTEA is the same as TEXT protocol > fields, and change the C code to return TEXT IN PG 14. Is tha

Re: Aw: Re: Minor documentation error regarding streaming replication protocol

2020-10-15 Thread Bruce Momjian
On Thu, Oct 15, 2020 at 11:18:33AM -0400, Tom Lane wrote: > Bruce Momjian writes: > > On Thu, Oct 15, 2020 at 10:03:46AM -0400, Tom Lane wrote: > >> I don't really see why our only documentation choices are "text" or > >> "bytea". Can't we just write "(raw file contents)" or the like? > > > Agre

Re: Aw: Re: Minor documentation error regarding streaming replication protocol

2020-10-15 Thread Tom Lane
Bruce Momjian writes: > On Thu, Oct 15, 2020 at 10:03:46AM -0400, Tom Lane wrote: >> I don't really see why our only documentation choices are "text" or >> "bytea". Can't we just write "(raw file contents)" or the like? > Agreed. The reason they are those labels is that the C code has to > labe

Re: Aw: Re: Minor documentation error regarding streaming replication protocol

2020-10-15 Thread Bruce Momjian
On Thu, Oct 15, 2020 at 10:03:46AM -0400, Tom Lane wrote: > Brar Piening writes: > > Bruce Momjian wrote: > >> I did some more research on this. It turns out timeline 'content' is > >> the only BYTEA listed in the protocol docs, even though it just passes C > >> strings to pq_sendbytes(), just li

Re: Aw: Re: Minor documentation error regarding streaming replication protocol

2020-10-15 Thread Fujii Masao
On 2020/10/15 23:03, Tom Lane wrote: Brar Piening writes: Bruce Momjian wrote: I did some more research on this. It turns out timeline 'content' is the only BYTEA listed in the protocol docs, even though it just passes C strings to pq_sendbytes(), just like many other fields like the field

Re: Aw: Re: Minor documentation error regarding streaming replication protocol

2020-10-15 Thread Tom Lane
Brar Piening writes: > Bruce Momjian wrote: >> I did some more research on this. It turns out timeline 'content' is >> the only BYTEA listed in the protocol docs, even though it just passes C >> strings to pq_sendbytes(), just like many other fields like the field >> above it, the timeline histor

Aw: Re: Minor documentation error regarding streaming replication protocol

2020-10-15 Thread Brar Piening
Bruce Momjian wrote: >> Good point. The reporter was assuming the data would come to the client >> in the bytea output format specified by the GUC, e.g. \x..., so that >> doesn't happen either. As I said before, it is more raw bytes, but we >> don't have an SQL data type for that. > I did some mo