Postgres Pro
Downloads
Home   >   mailing lists

Thread: Is this a feature?

Is this a feature?

From
"Joshua D. Drake"
Date:
Hello,

It is past feature freeze which means we can't introduce new features. 
It is possible to submit a patch for slightly different logging output?

Take the following:

INFO:  analyzing "pg_catalog.pg_authid"
INFO:  "pg_authid": scanned 1 of 1 pages, containing 5 live rows and 0 
dead rows; 5 rows in sample, 5 estimated total rows

The above is completely redundant. Why not just say:

INFO:  "pg_authid": scanned 1 of 1 pages, containing 5 live rows and 0 
dead rows; 5 rows in sample, 5 estimated total rows

If the first line is meant to be an indicator, then make the above line 
do this:

INFO: analyzing "pg_catalog.pg_authid" :

Don't add a new line, and when the next step of information comes up 
append it to the existing line to get:

INFO: analyzing "pg_catalog.pg_authid": scanned 1 of 1 pages, containing 
5 live rows and 0 dead rows; 5 rows in sample, 5 estimated total rows

Sincerely,

Joshua D. Drake


-- 
      === The PostgreSQL Company: Command Prompt, Inc. ===
Sales/Support: +1.503.667.4564 || 24x7/Emergency: +1.800.492.2240
Providing the most comprehensive  PostgreSQL solutions since 1997             http://www.commandprompt.com/

Donate to the PostgreSQL Project: http://www.postgresql.org/about/donate
PostgreSQL Replication: http://www.commandprompt.com/products/

Re: Is this a feature?

From
"Florian G. Pflug"
Date:
Joshua D. Drake wrote:
> Take the following:
> 
> INFO:  analyzing "pg_catalog.pg_authid"
> INFO:  "pg_authid": scanned 1 of 1 pages, containing 5 live rows and 0 
> dead rows; 5 rows in sample, 5 estimated total rows
> 
> The above is completely redundant. Why not just say:
> 
> INFO:  "pg_authid": scanned 1 of 1 pages, containing 5 live rows and 0 
> dead rows; 5 rows in sample, 5 estimated total rows
> 
> If the first line is meant to be an indicator, then make the above line 
> do this:
> 
> INFO: analyzing "pg_catalog.pg_authid" :
> 
> Don't add a new line, and when the next step of information comes up 
> append it to the existing line to get:
> 
> INFO: analyzing "pg_catalog.pg_authid": scanned 1 of 1 pages, containing 
> 5 live rows and 0 dead rows; 5 rows in sample, 5 estimated total rows

But then the line could only be pushed to the client *after* the analysis
of the table has finished, while with the current output you know what
postgres is currently doing, because you get "analyzing ..." *before*
the operation starts.

greetings, Florian Pflug