Add some sanity checks in executor for query ID reporting This commit adds three sanity checks in code paths of the executor where it is possible to use hooks, checking that a query ID is reported in pg_stat_activity if compute_query_id is enabled: - ExecutorRun() - ExecutorFinish() - ExecutorEnd()
This causes the test in pg_stat_statements added in 933848d16dc9 to complain immediately in ExecutorRun(). The idea behind this commit is to help extensions to detect if they are missing query ID reports when a query goes through the executor. Perhaps this will prove to be a bad idea, but let's see where this experience goes in v18 and newer versions. Reviewed-by: Sami Imseih Discussion: https://postgr.es/m/zujb5xckhh0a9...@paquier.xyz Branch ------ master Details ------- https://git.postgresql.org/pg/commitdiff/24f5205948093a96edf8213294b3d585ac3fe1fb Modified Files -------------- src/backend/executor/execMain.c | 10 ++++++++++ 1 file changed, 10 insertions(+)
