pg.ddx.io pgsql-hackers@postgresql.org mailing list archive
help / color / mirror / Atom feed installcheck fails when compute_query_id=on or pg_stat_statsement is loaded
3+ messages / 3 participants
[nested] [flat]
* installcheck fails when compute_query_id=on or pg_stat_statsement is loaded
@ 2021-10-15 07:36 Мельников Антон Андреевич <aamelnikov@inbox.ru>
2021-10-15 08:35 ` Re: installcheck fails when compute_query_id=on or pg_stat_statsement is loaded Julien Rouhaud <rjuju123@gmail.com>
0 siblings, 1 reply; 3+ messages in thread
From: Мельников Антон Андреевич @ 2021-10-15 07:36 UTC (permalink / raw)
To: pgsql-hackers
Hello!
When extension pg_stat_statsement is loaded into memory or compute_query_id=on in postgesql.conf
many of installcheck tests give errors.
The thing is that in *.out files appear lines "queryid = xxxxx" where xxxxx - some numeric value.
So 24 of 209 installcheck tests will fail.
It seems to be a normal behaviour as queryid calculation was moved into core in
https://git.postgresql.org/gitweb/?p=postgresql.git;a=commit;h=5fd9dfa5f5
but tests say that something wrong.
Create test variants is not the way, as queryid value varies randomly
from test to test at the same enviroment.
I think this is a problem because these fake errors can mask a real errors in relevant tests.
What’s your opinion?
Best regards,
Anton Melnikov
Postgres Professional: http://www.postgrespro.com
The Russian Postgres Company
^ permalink raw reply [nested|flat] 3+ messages in thread
* Re: installcheck fails when compute_query_id=on or pg_stat_statsement is loaded
2021-10-15 07:36 installcheck fails when compute_query_id=on or pg_stat_statsement is loaded Мельников Антон Андреевич <aamelnikov@inbox.ru>
@ 2021-10-15 08:35 ` Julien Rouhaud <rjuju123@gmail.com>
2021-10-15 13:46 ` Re: installcheck fails when compute_query_id=on or pg_stat_statsement is loaded Tom Lane <tgl@sss.pgh.pa.us>
0 siblings, 1 reply; 3+ messages in thread
From: Julien Rouhaud @ 2021-10-15 08:35 UTC (permalink / raw)
To: Мельников Антон Андреевич <aamelnikov@inbox.ru>; +Cc: pgsql-hackers
On Fri, Oct 15, 2021 at 3:36 PM Мельников Антон Андреевич
<aamelnikov@inbox.ru> wrote:
>
> When extension pg_stat_statsement is loaded into memory or compute_query_id=on in postgesql.conf
> many of installcheck tests give errors.
> [...]
> I think this is a problem because these fake errors can mask a real errors in relevant tests.
> What’s your opinion?
This has been discussed previously (although I can't find the thread
right now). Note that you don't really need to enable
pg_stat_statements, enabling compute_query_id is enough. The query
identifier is only displayed for EXPLAIN (VERBOSE), so it's already a
bit filtered. I don't see any simple way to entirely avoid the
problem though.
There are already many options that can break the regression tests, so
maybe it's ok to accept that this is yet another one. If not, the
only alternative I see is to add a boolean QUERY_ID option to EXPLAIN
and make sure that all tests use it, but it seems like a big hammer,
error prone, for a maybe small problem.
^ permalink raw reply [nested|flat] 3+ messages in thread
* Re: installcheck fails when compute_query_id=on or pg_stat_statsement is loaded
2021-10-15 07:36 installcheck fails when compute_query_id=on or pg_stat_statsement is loaded Мельников Антон Андреевич <aamelnikov@inbox.ru>
2021-10-15 08:35 ` Re: installcheck fails when compute_query_id=on or pg_stat_statsement is loaded Julien Rouhaud <rjuju123@gmail.com>
@ 2021-10-15 13:46 ` Tom Lane <tgl@sss.pgh.pa.us>
0 siblings, 0 replies; 3+ messages in thread
From: Tom Lane @ 2021-10-15 13:46 UTC (permalink / raw)
To: Julien Rouhaud <rjuju123@gmail.com>; +Cc: Мельников Антон Андреевич <aamelnikov@inbox.ru>; pgsql-hackers
Julien Rouhaud <rjuju123@gmail.com> writes:
> ... Note that you don't really need to enable
> pg_stat_statements, enabling compute_query_id is enough. The query
> identifier is only displayed for EXPLAIN (VERBOSE), so it's already a
> bit filtered. I don't see any simple way to entirely avoid the
> problem though.
> There are already many options that can break the regression tests, so
> maybe it's ok to accept that this is yet another one.
Yeah, that's my reaction. We could add "compute_query_id = off" to
the database-level settings set up by pg_regress, but that cure seems
worse than the disease. It would make it impossible to run the
regression tests with pg_stat_statements loaded, which you might wish
to do (just ignoring the bogus test failures) as a way of testing
pg_stat_statements.
regards, tom lane
^ permalink raw reply [nested|flat] 3+ messages in thread
end of thread, other threads:[~2021-10-15 13:46 UTC | newest]
Thread overview: 3+ messages (download: mbox mbox.gz follow: Atom feed)
-- links below jump to the message on this page --
2021-10-15 07:36 installcheck fails when compute_query_id=on or pg_stat_statsement is loaded Мельников Антон Андреевич <aamelnikov@inbox.ru>
2021-10-15 08:35 ` Julien Rouhaud <rjuju123@gmail.com>
2021-10-15 13:46 ` Tom Lane <tgl@sss.pgh.pa.us>
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