Show parameters of CALL as constants in pg_stat_statements
commit11c34b342bd77a292b625c32b70b44c02db8f3ed
authorMichael Paquier <michael@paquier.xyz>
Thu, 28 Sep 2023 06:17:55 +0000 (28 15:17 +0900)
committerMichael Paquier <michael@paquier.xyz>
Thu, 28 Sep 2023 06:17:55 +0000 (28 15:17 +0900)
tree74dabe1f42525f0c57f096a8f664438b3ac57967
parentd060e921ea5aa47b6265174c32e1128cebdbc3df
Show parameters of CALL as constants in pg_stat_statements

This commit changes the query jumbling of CallStmt so as its IN/OUT
parameters are able to show up as constants with a parameter symbol in
pg_stat_statements, like:
CALL proc1($1, $2);
CALL proc2($1, $2, $3);

The transformed FuncExpr is used in the query ID computation instead of
the FuncCall generated by the parser, so as it is sensitive to the OID
of the procedure and its list of input arguments.  The output arguments
are handled in a separate list in CallStmt, which is also included in
the computation.

Tests are added to pg_stat_statements to show how this affects CALL with
IN/OUT parameters as well as overloaded functions.

Like 638d42a3c520 or 31de7e60da34, this improves the monitoring of
workloads with a lot of CALL statements, preventing unnecessary bloat
when these use different input (or event output) values.

Author: Sami Imseih
Discussion: https://postgr.es/m/B44FA29D-EBD0-4DD9-ABC2-16F1CB087074@amazon.com
contrib/pg_stat_statements/expected/utility.out
contrib/pg_stat_statements/sql/utility.sql
src/include/nodes/parsenodes.h