Re: New GUC to sample log queries

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

In response to

Responses

Browse pgsql-hackers by date

  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