From: | Justin Pryzby <pryzby(at)telsasoft(dot)com> |
---|---|
To: | Andres Freund <andres(at)anarazel(dot)de> |
Cc: | Melanie Plageman <melanieplageman(at)gmail(dot)com>, Kyotaro Horiguchi <horikyota(dot)ntt(at)gmail(dot)com>, tgl(at)sss(dot)pgh(dot)pa(dot)us, vignesh21(at)gmail(dot)com, lukas(at)fittl(dot)com, alvherre(at)alvh(dot)no-ip(dot)org, magnus(at)hagander(dot)net, pgsql-hackers(at)postgresql(dot)org, thomas(dot)munro(at)gmail(dot)com, m(dot)sakrejda(at)gmail(dot)com |
Subject: | Re: pg_stat_bgwriter.buffers_backend is pretty meaningless (and more?) |
Date: | 2023-03-09 12:51:31 |
Message-ID: | ZAnWU8WbXEDjrfUE@telsasoft.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Tue, Mar 07, 2023 at 10:18:44AM -0800, Andres Freund wrote:
> Hi,
>
> On 2023-03-06 15:21:14 -0500, Melanie Plageman wrote:
> > Good point. Attached is what you suggested. I committed the transaction
> > before the drop table so that the statistics would be visible when we
> > queried pg_stat_io.
>
> Pushed, thanks for the report, analysis and fix, Tom, Horiguchi-san, Melanie.
There's a 2nd portion of the test that's still flapping, at least on
cirrusci.
The issue that Tom mentioned is at:
SELECT :io_sum_shared_after_writes > :io_sum_shared_before_writes;
But what I've seen on cirrusci is at:
SELECT :io_sum_shared_after_writes > :io_sum_shared_before_writes;
https://api.cirrus-ci.com/v1/artifact/task/6701069548388352/log/src/test/recovery/tmp_check/regression.diffs
https://api.cirrus-ci.com/v1/artifact/task/5355168397524992/log/src/test/recovery/tmp_check/regression.diffs
https://api.cirrus-ci.com/v1/artifact/task/6142435751886848/testrun/build/testrun/recovery/027_stream_regress/log/regress_log_027_stream_regress
It'd be neat if cfbot could show a histogram of test failures, although
I'm not entirely sure what granularity would be most useful: the test
that failed (027_regress) or the way it failed (:after_write >
:before_writes). Maybe it's enough to show the test, with links to its
recent failures.
--
Justin
From | Date | Subject | |
---|---|---|---|
Next Message | Nazir Bilal Yavuz | 2023-03-09 13:02:44 | Re: Refactor calculations to use instr_time |
Previous Message | Drouvot, Bertrand | 2023-03-09 12:24:32 | Re: Avoid multiple SetLatch() calls in procsignal_sigusr1_handler() |