Re: compute_query_id and pg_stat_statements
Stephen Frost <sfrost@snowman.net>
From: Stephen Frost <sfrost@snowman.net>
To: Tom Lane <tgl@sss.pgh.pa.us>
Cc: Bruce Momjian <bruce@momjian.us>, Christoph Berg <myon@debian.org>, Julien Rouhaud <rjuju123@gmail.com>, Pavel Stehule <pavel.stehule@gmail.com>, Kyotaro Horiguchi <horikyota.ntt@gmail.com>, Magnus Hagander <magnus@hagander.net>, Fujii Masao <masao.fujii@oss.nttdata.com>, Michael Paquier <michael@paquier.xyz>, Andres Freund <andres@anarazel.de>, Alvaro Herrera <alvherre@alvh.no-ip.org>, Peter Eisentraut <peter.eisentraut@enterprisedb.com>, PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Date: 2021-05-13T18:47:23Z
Lists: pgsql-hackers
Greetings, * Tom Lane (tgl@sss.pgh.pa.us) wrote: > Stephen Frost <sfrost@snowman.net> writes: > > There's a ridiculously simple option here which is: drop the idea that > > we support an extension redefining the query id and then just make it > > on/off with the default to be 'on'. > > I do not think that defaulting it to 'on' is acceptable unless you can > show that the added overhead is negligible. IIUC the measurements that > have been done show the opposite. Ah, right, it had only been done before when pg_stat_statements was loaded.. In which case, it seems like we should: a) go back to that b) if someone wants an alternative query ID, tell them to add it to pg_stat_statements and make it configurable *there* c) Have pg_stat_statements provide another function/view/etc that folks can use to get a queryid for an ongoing query ..? d) Maybe come up with a way for extensions, generically, to make a value available to log_line_prefix? That could be pretty interesting. Or just accept that this is a bit hokey with the 'auto' approach. I get Bruce has concerns about it but I'm not convinced that it's actually all that bad to go with that. Thanks, Stephen
Commits
-
Allow compute_query_id to be set to 'auto' and make it default
- cafde58b337e 14.0 landed
-
Move pg_stat_statements query jumbling to core.
- 5fd9dfa5f50e 14.0 cited