Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                
Skip to content

Commit a9a8108

Browse files
committed
Remove test for pg_stat_reset_shared() in stats.sql
Some buildfarm members have been failing a test related to pg_stat_io, as an effect of 23c8c0c that has switched pg_stat_reset_shared() from being a no-op to reset all shared stats types. This extra reset has the effect to make pg_stat_io's counters low enough that little concurrent activity is enough to cause a failure. Another thing I have considered is to move this sequence at the end of stats.sql, but there are other instabilities, one being pg_stat_wal. Knowing that there are already tests for the reset of each individual shared stats target, this test has limited value, so let's remove it to minimize the number of resets done for each shared stats type. This should hopefully improve the stability of the whole. Discussion: https://postgr.es/m/3500949.1700935734@sss.pgh.pa.us
1 parent 5ad4932 commit a9a8108

File tree

2 files changed

+0
-54
lines changed

2 files changed

+0
-54
lines changed

src/test/regress/expected/stats.out

-44
Original file line numberDiff line numberDiff line change
@@ -990,50 +990,6 @@ SELECT stats_reset > :'wal_reset_ts'::timestamptz FROM pg_stat_wal;
990990
(1 row)
991991

992992
SELECT stats_reset AS wal_reset_ts FROM pg_stat_wal \gset
993-
-- Test that reset_shared with no argument resets all the stats types
994-
-- supported (providing NULL as argument has the same effect).
995-
SELECT pg_stat_reset_shared();
996-
pg_stat_reset_shared
997-
----------------------
998-
999-
(1 row)
1000-
1001-
SELECT stats_reset > :'archiver_reset_ts'::timestamptz FROM pg_stat_archiver;
1002-
?column?
1003-
----------
1004-
t
1005-
(1 row)
1006-
1007-
SELECT stats_reset > :'bgwriter_reset_ts'::timestamptz FROM pg_stat_bgwriter;
1008-
?column?
1009-
----------
1010-
t
1011-
(1 row)
1012-
1013-
SELECT stats_reset > :'checkpointer_reset_ts'::timestamptz FROM pg_stat_checkpointer;
1014-
?column?
1015-
----------
1016-
t
1017-
(1 row)
1018-
1019-
SELECT stats_reset > :'recovery_prefetch_reset_ts'::timestamptz FROM pg_stat_recovery_prefetch;
1020-
?column?
1021-
----------
1022-
t
1023-
(1 row)
1024-
1025-
SELECT max(stats_reset) > :'slru_reset_ts'::timestamptz FROM pg_stat_slru;
1026-
?column?
1027-
----------
1028-
t
1029-
(1 row)
1030-
1031-
SELECT stats_reset > :'wal_reset_ts'::timestamptz FROM pg_stat_wal;
1032-
?column?
1033-
----------
1034-
t
1035-
(1 row)
1036-
1037993
-- Test error case for reset_shared with unknown stats type
1038994
SELECT pg_stat_reset_shared('unknown');
1039995
ERROR: unrecognized reset target: "unknown"

src/test/regress/sql/stats.sql

-10
Original file line numberDiff line numberDiff line change
@@ -494,16 +494,6 @@ SELECT pg_stat_reset_shared('wal');
494494
SELECT stats_reset > :'wal_reset_ts'::timestamptz FROM pg_stat_wal;
495495
SELECT stats_reset AS wal_reset_ts FROM pg_stat_wal \gset
496496

497-
-- Test that reset_shared with no argument resets all the stats types
498-
-- supported (providing NULL as argument has the same effect).
499-
SELECT pg_stat_reset_shared();
500-
SELECT stats_reset > :'archiver_reset_ts'::timestamptz FROM pg_stat_archiver;
501-
SELECT stats_reset > :'bgwriter_reset_ts'::timestamptz FROM pg_stat_bgwriter;
502-
SELECT stats_reset > :'checkpointer_reset_ts'::timestamptz FROM pg_stat_checkpointer;
503-
SELECT stats_reset > :'recovery_prefetch_reset_ts'::timestamptz FROM pg_stat_recovery_prefetch;
504-
SELECT max(stats_reset) > :'slru_reset_ts'::timestamptz FROM pg_stat_slru;
505-
SELECT stats_reset > :'wal_reset_ts'::timestamptz FROM pg_stat_wal;
506-
507497
-- Test error case for reset_shared with unknown stats type
508498
SELECT pg_stat_reset_shared('unknown');
509499

0 commit comments

Comments
 (0)