Show savepoint names as constants in pg_stat_statements
authorMichael Paquier <[email protected]>
Thu, 27 Jul 2023 00:42:33 +0000 (09:42 +0900)
committerMichael Paquier <[email protected]>
Thu, 27 Jul 2023 00:42:33 +0000 (09:42 +0900)
commit31de7e60da34761365f94dc76cc2c1bf2172d1bc
tree4220dce9f328eecccdcf04d0120b9d44d5b0983e
parent19c590f6a795831b34782a4d69fa6a52dc9d03c3
Show savepoint names as constants in pg_stat_statements

In pg_stat_statements, savepoint names now show up as constants with a
parameter symbol, using as base query string the one added as a new
entry to the PGSS hash table, leading to:
RELEASE $1
ROLLBACK TO $1
SAVEPOINT $1

Applying constants to these query parts is a huge advantage for
workloads that generate randomly savepoint points, like ORMs (Django is
at the origin of this patch).  The ODBC driver is a second layer that
likes a lot savepoints, though it does not use a random naming pattern.

A "location" field is added to TransactionStmt, now set only for
savepoints.  The savepoint name is ignored by the query jumbling.  The
location can be extended to other query patterns, if required, like 2PC
commands.  Some tests are added to pg_stat_statements for all the query
patterns supported by the parser.

ROLLBACK, ROLLBACK TO SAVEPOINT and ROLLBACK TRANSACTION TO SAVEPOINT
have the same Node representation, so all these are equivalents.  The
same happens for RELEASE and RELEASE SAVEPOINT.

Author: Greg Sabino Mullane
Discussion: https://postgr.es/m/CAKAnmm+2s9PA4OaumwMJReWHk8qvJ_-g1WqxDRDAN1BSUfxyTw@mail.gmail.com
contrib/pg_stat_statements/expected/utility.out
contrib/pg_stat_statements/sql/utility.sql
src/backend/parser/gram.y
src/include/nodes/parsenodes.h