Re: Using XLogFileNameP in critical section - Mailing list pgsql-hackers

From Masahiko Sawada
Subject Re: Using XLogFileNameP in critical section
Date
Msg-id CA+fd4k6L9k=61iwOuk2R1h5UzX0aUds-1jgy-Apr9ZBKdHQNSA@mail.gmail.com
Whole thread Raw
In response to Re: Using XLogFileNameP in critical section  (Michael Paquier <michael@paquier.xyz>)
List pgsql-hackers
On Tue, 3 Dec 2019 at 07:09, Michael Paquier <michael@paquier.xyz> wrote:
>
> On Mon, Dec 02, 2019 at 10:14:58PM +0100, Masahiko Sawada wrote:
> > Agreed. I've attached the updated version patch. Please review it.
>
> Thanks, applied on HEAD after a few edits.  gettext() does not set
> errno, so the new style of issue_xlog_fsync() is actually fine.

Thanks!

> Please note that there was one mistake in the patch: you forgot to
> assign back errno in assign_xlog_sync_method() after generating the
> file name.

My bad. Thank you for fixing it.

Regards,

-- 
Masahiko Sawada            http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services



pgsql-hackers by date:

Previous
From: Masahiko Sawada
Date:
Subject: Re: [HACKERS] Block level parallel vacuum
Next
From: Alvaro Herrera
Date:
Subject: Re: log bind parameter values on error