Re: pg_dump reporing version of server & pg_dump as comments in the output

From: "Wang, Jing" <jingw(at)fast(dot)au(dot)fujitsu(dot)com>
To: Euler Taveira <euler(at)timbira(dot)com(dot)br>
Cc: <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: pg_dump reporing version of server & pg_dump as comments in the output
Date: 2014-03-04 05:58:20
Message-ID: F40B0968DB0A904DA78A924E633BE7863C28E9@SYDEXCHTMP2.au.fjanz.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 4 March 2014 2:41 Euler Taveira wrote:
>On 27-02-2014 21:10, Wang, Jing wrote:
>> Using pg_dump can dump the data into the file with format set to be
>> 'c','t' or plain text. In the existing version the version of server
&
>> pg_dump is already there when the format of file is 'c' or 't'. And
>> even for the plain text format file the version of server & pg_dump
is
>> already there if using '--verbose' in pg_dump. Using '--verbose'
leads
>> to some many other prints which are not required always.
>>
>I don't buy your argument. Why isn't verbose option sufficient? Did you
read the old thread about this [1]?
>[1] http://www.postgresql.org/message-id/3677.1253912361@sss.pgh.pa.us

>AFAICS a lot of people compare pg_dump diffs. If we apply this patch,
it would break those applications. Also, it is *already* available if
you add verbose option (which is sufficient to satisfy those that want
the client and/or
>server version) in plain mode (the other modes already include the
desired info by default). In the past, timestamps were removed to avoid
noise in diffs.

Sorry, I don't understand which application will break? Can you give me
more detail information?
Timestamps always vary which do affect the diff. But I can't imagine
why adding the version will affect the pg_dump diffs.
I don't think the version number vary frequently.

Kind regards
Jing Wang
Fujitsu Australia

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Andres Freund 2014-03-04 06:31:21 Re: GSoC proposal - "make an unlogged table logged"
Previous Message Fabrízio de Royes Mello 2014-03-04 05:35:54 Re: psql: show only failed queries