| From: | "Imseih (AWS), Sami" <simseih(at)amazon(dot)com> |
|---|---|
| To: | Michael Paquier <michael(at)paquier(dot)xyz>, torikoshia <torikoshia(at)oss(dot)nttdata(dot)com> |
| Cc: | Julien Rouhaud <rjuju123(at)gmail(dot)com>, Justin Pryzby <pryzby(at)telsasoft(dot)com>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: Record queryid when auto_explain.log_verbose is on |
| Date: | 2023-03-06 23:50:08 |
| Message-ID: | 33C39727-0BE6-4691-914A-A7CF39995A58@amazon.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
I am wondering if this patch should be backpatched?
The reason being is in auto_explain documentation [1],
there is a claim of equivalence of the auto_explain.log_verbose
option and EXPLAIN(verbose)
"..... it's equivalent to the VERBOSE option of EXPLAIN."
This can be quite confusing for users of the extension.
The documentation should either be updated or a backpatch
all the way down to 14, which the version the query identifier
was moved to core. I am in favor of the latter.
Any thoughts?
[1] https://www.postgresql.org/docs/14/auto-explain.html
Regards,
--
Sami Imseih
Amazon Web Services (AWS)
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Stephen Frost | 2023-03-07 00:00:38 | Re: a very minor bug and a couple of comment changes for basebackup.c |
| Previous Message | Regina Obe | 2023-03-06 23:26:59 | RE: Ability to reference other extensions by schema in extension scripts |