Issue in recent pg_stat_statements?

From: David Christensen <david(dot)christensen(at)crunchydata(dot)com>
To: pgsql-hackers(at)postgresql(dot)org
Subject: Issue in recent pg_stat_statements?
Date: 2021-04-26 15:14:59
Message-ID: CAOxo6XJEYunL71g0yD-zRzNRRqBG0Ssw-ARygy5pGRdSjK5YLQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

-hackers,

So in doing some recent work on pg_stat_statements, I notice that while the
regression test still passes on HEAD, it appears that 4f0b096 (per git
bisect) changed/broke how this works compared to historical versions.

Essentially, when doing a fresh install of pg_stat_statements on a new
fresh db (outside of the regression framework), it's not returning any rows
from the view. I didn't see any related documentation changes, so as far
as I know, this should still be recording all statements as per normal.

My full steps to reproduce from a clean Centos 7 install are attached. I
have also been able to reproduce this on OS X and Fedora 33. The TL;DR is:

CREATE EXTENSION pg_stat_statements;
CREATE TABLE foo (a int, b text);
INSERT INTO foo VALUES (1,'a');
SELECT * FROM foo;
SELECT * FROM pg_stat_statements; -- returns nothing

Settings for pg_stat_statements:
postgres=# select name, setting from pg_settings where name like
'pg_stat_statements%';
name | setting
-----------------------------------+---------
pg_stat_statements.max | 5000
pg_stat_statements.save | on
pg_stat_statements.track | top
pg_stat_statements.track_planning | off
pg_stat_statements.track_utility | on
(5 rows)

Is this an expected change, or is this in fact broken? In previous
revisions, this was showing the INSERT and SELECT at the very least. I'm
unclear as to why the regression test is still passing, so want to verify
that I'm not doing something wrong in the testing.

Best,

David

Attachment Content-Type Size
pg_stat_statements-error.sh text/x-sh 1.1 KB

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Magnus Hagander 2021-04-26 15:18:09 Re: Issue in recent pg_stat_statements?
Previous Message Tom Lane 2021-04-26 14:55:42 Re: Does rewriteTargetListIU still need to add UPDATE tlist entries?