Re: New GUC to sample log queries
Alvaro Herrera <alvherre@2ndquadrant.com>
From: Alvaro Herrera <alvherre@2ndquadrant.com>
To: Adrien Nayrat <adrien.nayrat@anayrat.info>
Cc: Thomas Munro <thomas.munro@enterprisedb.com>, Dmitry Dolgov <9erthalion6@gmail.com>, Tomas Vondra <tomas.vondra@2ndquadrant.com>, vik.fearing@2ndquadrant.com, Robert Haas <robertmhaas@gmail.com>, Michael Paquier <michael@paquier.xyz>, David Rowley <david.rowley@2ndquadrant.com>, Pg Hackers <pgsql-hackers@postgresql.org>
Date: 2018-11-29T13:27:07Z
Lists: pgsql-hackers
Attachments
- 0001-pgindent.patch (text/x-diff)
=# 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
Commits
-
Use pg_strong_random() to select each server process's random seed.
- 4203842a1cd0 12.0 landed
-
Use a separate random seed for SQL random()/setseed() functions.
- 6645ad6bdd81 12.0 landed
-
Marginal performance hacking in erand48.c.
- 6b9bba2df8d4 12.0 landed
-
Fix latent problem with pg_jrand48().
- e09046641114 12.0 landed
- f256995e33d2 10.7 landed
- d58e01f8abe2 11.2 landed
-
Silence compiler warning
- 9dc122585551 12.0 landed
-
Add log_statement_sample_rate parameter
- 88bdbd3f7460 12.0 landed