Re: Use XLOG_CONTROL_FILE macro everywhere? - Mailing list pgsql-hackers

From Anton A. Melnikov
Subject Re: Use XLOG_CONTROL_FILE macro everywhere?
Date
Msg-id cfaa1778-99f2-4348-8ab7-56be503e6d39@postgrespro.ru
Whole thread Raw
In response to Re: Use XLOG_CONTROL_FILE macro everywhere?  (Fujii Masao <masao.fujii@oss.nttdata.com>)
Responses Re: Use XLOG_CONTROL_FILE macro everywhere?
List pgsql-hackers
Hi!

On 04.04.2025 19:32, Fujii Masao wrote:
> I'm not sure there's clear consensus yet on the changes in the 0001 and
> 0002 patches, and it might not be worth rushing them in right before
> the feature freeze. So for now, I reviewed and updated only the 0003 patch,
> since there seems to be agreement on that one.
> 
> I've attached the updated version. It fixes a typo and replaces
> the remaining hardcoded control file name with the XLOG_CONTROL_FILE macro.

Thanks! Looks good for me.
Maybe extend this to perl tests since there are several hardcoded names too?
The patch attached tries to do this.

> How about committing the attached patch first? We can consider applying
> the others later if consensus is reached

Yes, of cause. IMO, this is the best way now.


Best wishes,


-- 
Anton A. Melnikov
Postgres Professional: http://www.postgrespro.com
The Russian Postgres Company
Attachment

pgsql-hackers by date:

Previous
From: Andres Freund
Date:
Subject: Re: BTScanOpaqueData size slows down tests
Next
From: Corey Huinker
Date:
Subject: Re: Statistics Import and Export