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 13:27:07
Message-ID: 20181129132707.ybvbqyeeelvp5noq@alvherre.pgsql
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers


=# 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 2 ]────────────────────────────────────────────────────────────────────────────────────────────────────
name │ log_min_error_statement
short_desc │ Causes all statements generating error at or above this level to be logged.
extra_desc │ Each level includes all the levels that follow it. The later the level, the fewer messages are sent.
category │ Reporting and Logging / When to Log
─[ RECORD 3 ]────────────────────────────────────────────────────────────────────────────────────────────────────
name │ log_min_messages
short_desc │ Sets the message levels that are logged.
extra_desc │ Each level includes all the levels that follow it. The later the level, the fewer messages are sent.
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

The description here seems a bit short on details to me. I would say
something like "Fraction of statements over log_min_duration_statement
to log"; otherwise it's not clear why this doesn't apply to
log_statement. I think the extra_desc should be more verbose too. (Not
really clear to me what to put in each ... suggestions welcome.)

(More generally, I think we should add a lot more juice to the GUC
description fields.)

Attached is pgindent fixes for your next submission.

--
Álvaro Herrera https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services

Attachment Content-Type Size
0001-pgindent.patch text/x-diff 2.0 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Dmitry Dolgov 2018-11-29 13:27:50 Re: [WIP] [B-Tree] Retail IndexTuple deletion
Previous Message Dmitry Dolgov 2018-11-29 13:13:30 Re: Usage of epoch in txid_current