| From: | torikoshia <torikoshia(at)oss(dot)nttdata(dot)com> |
|---|---|
| To: | Matthias van de Meent <boekewurm+postgres(at)gmail(dot)com>, Pgsql Hackers <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: RFC: Logging plan of the running query |
| Date: | 2021-05-13 08:23:33 |
| Message-ID: | ce86e4f72f09d5497e8ad3a162861d33@oss.nttdata.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Thank you all for your positive comments.
On 2021-05-12 21:55, Matthias van de Meent wrote:
> Great idea. One feature I'd suggest would be adding a 'format' option
> as well, if such feature would be feasable.
Thanks for the comment!
During the development of pg_log_backend_memory_contexts(), I tried to
make the number of contexts to record configurable by making it GUC
variable or putting it on the shared memory, but the former seemed an
overkill and the latter introduced some ugly behaviors, so we decided
to make it a static number[1].
I think we face the same difficulty here.
Allowing to select the format would be better as auto_explain does by
auto_explain.log_format, but I'm a bit doubtful that it is worth the
costs.
Regards,
--
Atsushi Torikoshi
NTT DATA CORPORATION
| From | Date | Subject | |
|---|---|---|---|
| Next Message | torikoshia | 2021-05-13 08:26:20 | Re: RFC: Logging plan of the running query |
| Previous Message | David Rowley | 2021-05-13 08:20:36 | Re: Executor code - found an instance of a WHILE that should just be an IF |