pg.ddx.io  pgsql-committers@postgresql.org mailing list archive  
help / color / mirror / Atom feed
From: Michael Paquier <michael@paquier.xyz>
To: pgsql-committers@lists.postgresql.org
Subject: pgsql: Propagate query IDs of utility statements in functions
Date: Fri, 19 Jul 2024 01:21:58 +0000
Message-ID: <E1sUcJq-000CZ4-LR@gemulon.postgresql.org> (raw)

Propagate query IDs of utility statements in functions

For utility statements defined within a function, the query tree is
copied to a PlannedStmt as utility commands do not require planning.
However, the query ID was missing from the information passed down.

This leads to plugins relying on the query ID like pg_stat_statements to
not be able to track utility statements within function calls.  Tests
are added to check this behavior, depending on pg_stat_statements.track.

This is an old bug.  Now, query IDs for utilities are compiled using
their parsed trees rather than the query string since v16
(3db72ebcbe20), leading to less bloat with utilities, so backpatch down
only to this version.

Author: Anthonin Bonnefoy
Discussion: https://postgr.es/m/CAO6_XqrGp-uwBqi3vBPLuRULKkddjC7R5QZCgsFren=8E+m2Sg@mail.gmail.com
Backpatch-through: 16

Branch
------
REL_17_STABLE

Details
-------
https://git.postgresql.org/pg/commitdiff/38e271d3c2e8c3bfa4d57fd81c3e47b40f1f5eb3

Modified Files
--------------
.../pg_stat_statements/expected/level_tracking.out | 45 ++++++++++++++++++++++
contrib/pg_stat_statements/sql/level_tracking.sql  | 22 +++++++++++
src/backend/executor/functions.c                   |  1 +
3 files changed, 68 insertions(+)



view thread (3+ messages)

Message-ID: <E1sUcJq-000CZ4-LR@gemulon.postgresql.org>
Permalink:  ../E1sUcJq-000CZ4-LR@gemulon.postgresql.org/
Also on:    postgresql.org/message-id/E1sUcJq-000CZ4-LR@gemulon.postgresql.org

 · 

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-committers@postgresql.org
  Cc: michael@paquier.xyz, pgsql-committers@lists.postgresql.org
  Subject: Re: pgsql: Propagate query IDs of utility statements in functions
  In-Reply-To: <E1sUcJq-000CZ4-LR@gemulon.postgresql.org>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox