From: | Bharath Rupireddy <bharath(dot)rupireddyforpostgres(at)gmail(dot)com> |
---|---|
To: | "Hayato Kuroda (Fujitsu)" <kuroda(dot)hayato(at)fujitsu(dot)com> |
Cc: | Amit Kapila <amit(dot)kapila16(at)gmail(dot)com>, vignesh C <vignesh21(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
Subject: | Re: A failure in t/038_save_logical_slots_shutdown.pl |
Date: | 2024-01-25 10:57:20 |
Message-ID: | CALj2ACWLbXzyF5sC7fYAekk+9nev=Y0Z+XE0SUNXkD-R9B1tuA@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Tue, Jan 16, 2024 at 12:13 PM Hayato Kuroda (Fujitsu)
<kuroda(dot)hayato(at)fujitsu(dot)com> wrote:
>
> Dear Amit, Bharath,
>
> > This is a more strict check because it is possible that even if the
> > latest confirmed_flush location is not persisted there is no
> > meaningful decodable WAL between whatever the last confirmed_flush
> > location saved on disk and the shutdown_checkpoint record.
> > Kuroda-San/Vignesh, do you have any suggestion on this one?
>
> I think it should be as testcase explicitly. There are two reasons:
>
> * e0b2eed is a commit for backend codes, so it should be tested by src/test/*
> files. Each src/bin/XXX/*.pl files should test only their executable.
> * Assuming that the feature would be broken. In this case 003_logical_slots.pl
> would fail, but we do not have a way to recognize on the build farm.
> 038_save_logical_slots_shutdown.pl helps to distinguish the case.
+1 to keep 038_save_logical_slots_shutdown.pl as-is.
> Based on that, I think it is OK to add advance_wal() and comments, like Bharath's patch.
Thanks. I'll wait a while and then add it to CF to not lose it in the wild.
--
Bharath Rupireddy
PostgreSQL Contributors Team
RDS Open Source Databases
Amazon Web Services: https://aws.amazon.com
From | Date | Subject | |
---|---|---|---|
Next Message | Przemysław Sztoch | 2024-01-25 11:14:38 | Re: UUID v7 |
Previous Message | Dilip Kumar | 2024-01-25 10:45:31 | Re: index prefetching |