| From: | "Gregory Stark (as CFM)" <stark(dot)cfm(at)gmail(dot)com> |
|---|---|
| To: | Nitin Jadhav <nitinjadhavpostgres(at)gmail(dot)com> |
| Cc: | Andres Freund <andres(at)anarazel(dot)de>, Robert Haas <robertmhaas(at)gmail(dot)com>, Pg Hackers <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: Inconsistency in reporting checkpointer stats |
| Date: | 2023-03-20 20:01:02 |
| Message-ID: | CAM-w4HPcKgcnPHAiQv6-3cECAM0H0havP2N5qRYddvkqXhaRNQ@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Sun, 19 Feb 2023 at 04:58, Nitin Jadhav
<nitinjadhavpostgres(at)gmail(dot)com> wrote:
>
> > This doesn't pass the tests, because the regression tests weren't adjusted:
> > https://api.cirrus-ci.com/v1/artifact/task/5937624817336320/testrun/build/testrun/regress/regress/regression.diffs
>
> Thanks for sharing this. I have fixed this in the patch attached.
AFAICS this patch was marked Waiting on Author due to this feedback on
Feb 14 but the patch was updated Feb 19 and is still passing today.
I've marked it Needs Review.
I'm not sure if all of the feedback from Bharath Rupireddy and Kyotaro
Horiguchi was addressed.
If there's any specific questions remaining you need feedback on it
would be good to ask explicitly. Otherwise if you think it's ready you
could mark it Ready for Commit.
--
Gregory Stark
As Commitfest Manager
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Gregory Stark (as CFM) | 2023-03-20 20:05:03 | Re: [PATCH] Add <<none>> support to sepgsql_restorecon |
| Previous Message | Gregory Stark (as CFM) | 2023-03-20 19:56:04 | Re: Optimizing Node Files Support |