| From: | Bertrand Drouvot <bertranddrouvot(dot)pg(at)gmail(dot)com> |
|---|---|
| To: | Álvaro Herrera <alvherre(at)kurilemu(dot)de> |
| Cc: | Peter Eisentraut <peter(at)eisentraut(dot)org>, Michael Paquier <michael(at)paquier(dot)xyz>, Heikki Linnakangas <hlinnaka(at)iki(dot)fi>, Quan Zongliang <quanzongliang(at)yeah(dot)net>, pgsql-hackers(at)lists(dot)postgresql(dot)org |
| Subject: | Re: Consistently use the XLogRecPtrIsInvalid() macro |
| Date: | 2025-11-07 17:22:59 |
| Message-ID: | aQ4q80lXyWnmUAX+@ip-10-97-1-34.eu-west-3.compute.internal |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Hi,
On Fri, Nov 07, 2025 at 03:03:03PM +0000, Bertrand Drouvot wrote:
> Hi,
>
> On Fri, Nov 07, 2025 at 02:37:32PM +0100, Álvaro Herrera wrote:
> > On 2025-Nov-07, Bertrand Drouvot wrote:
> >
> > > Agree, will modify the .cocci scripts that way.
> >
> > I just noticed that we missed this ... maybe you want to include it also?
> >
> > - MyProc->waitLSN = 0;
> > + MyProc->waitLSN = InvalidXLogRecPtr;
> >
> > - lastLSN = 0;
> > + lastLSN = InvalidXLogRecPtr;
> >
> > - MyProc->waitLSN = 0;
> > + MyProc->waitLSN = InvalidXLogRecPtr;
>
> Yeah, that's another story here that is worth to look at too. Will do.
What do you think of the attached? It contains the ones you mentioned and some
others. The patch attached has been generated by the .cocci script [1].
Regards,
--
Bertrand Drouvot
PostgreSQL Contributors Team
RDS Open Source Databases
Amazon Web Services: https://aws.amazon.com
| Attachment | Content-Type | Size |
|---|---|---|
| v8-0001-Replace-literal-0-with-InvalidXLogRecPtr-for-XLog.patch | text/x-diff | 8.3 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Bryan Green | 2025-11-07 17:28:48 | Re: [PATCH] O_CLOEXEC not honored on Windows - handle inheritance chain |
| Previous Message | Dagfinn Ilmari Mannsåker | 2025-11-07 17:18:41 | Re: Consistently use the XLogRecPtrIsInvalid() macro |