From: | Alvaro Herrera <alvherre(at)2ndquadrant(dot)com> |
---|---|
To: | Adrien NAYRAT <adrien(dot)nayrat(at)anayrat(dot)info> |
Cc: | Thomas Munro <thomas(dot)munro(at)enterprisedb(dot)com>, Dmitry Dolgov <9erthalion6(at)gmail(dot)com>, Tomas Vondra <tomas(dot)vondra(at)2ndquadrant(dot)com>, vik(dot)fearing(at)2ndquadrant(dot)com, Robert Haas <robertmhaas(at)gmail(dot)com>, Michael Paquier <michael(at)paquier(dot)xyz>, David Rowley <david(dot)rowley(at)2ndquadrant(dot)com>, Pg Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: New GUC to sample log queries |
Date: | 2018-11-29 17:35:15 |
Message-ID: | 20181129173515.t2rf6hjvzbw2ijra@alvherre.pgsql |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 2018-Nov-29, Adrien NAYRAT wrote:
> > =# select name, short_desc, extra_desc, category from pg_settings where category like 'Reporting%When%' ;
> > ─[ RECORD 1 ]────────────────────────────────────────────────────────────────────────────────────────────────────
> > name │ log_min_duration_statement
> > short_desc │ Sets the minimum execution time above which statements will be logged.
> > extra_desc │ Zero prints all queries. -1 turns this feature off.
> > category │ Reporting and Logging / When to Log
> > ─[ RECORD 4 ]────────────────────────────────────────────────────────────────────────────────────────────────────
> > name │ log_statement_sample_rate
> > short_desc │ Fraction of statements to log.
> > extra_desc │ 1.0 logs all statements.
> > category │ Reporting and Logging / When to Log
> I like your short_desc, so I propose:
>
> short_desc: "Fraction of statements over log_min_duration_statement to log"
> long_desc: "If we only want a sample, use a value between 0 (never log) and
> 1.0 (always log)"
Sounds good to me (we -> you, I suppose?). I'd tweak the extra_desc for
log_min_duration_statement too, because we no longer log "all" when the
rate is <1; maybe "Zero prints all queries, subject to
log_statement_sample_rate. -1 turns this feature off."
I just noticed we don't use "you" anywhere in the descs, except for this
one:
name │ log_hostname
short_desc │ Logs the host name in the connection logs.
extra_desc │ By default, connection logs only show the IP address of the connecting host. If you want them to show the host name you can turn this on, but depending on your host name resolution setup it might impose a non-negligible performance penalty.
category │ Reporting and Logging / What to Log
Probably not worth fussing about :-)
> If you agree with that, I will send another patch (I will complete
> postgresql.conf.sample with the same description).
+1
--
Álvaro Herrera https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
From | Date | Subject | |
---|---|---|---|
Next Message | Alvaro Herrera | 2018-11-29 17:47:41 | Re: [PROPOSAL] extend the object names to the qualified names in pg_stat_statements |
Previous Message | Dmitry Dolgov | 2018-11-29 17:21:34 | Re: Changing the autovacuum launcher scheduling; oldest table first algorithm |