pg.ddx.io  pgsql-bugs@postgresql.org mailing list archive  
help / color / mirror / Atom feed
From: Andrew Dunstan <andrew@dunslane.net>
To: Michael Paquier <michael@paquier.xyz>
To: Alexandra Wang <alexandra.wang.oss@gmail.com>
Cc: Chris Travers <chris.travers@stormatics.tech>
Cc: Farooq Rashed <farooq.rashed@desc.gov.ae>
Cc: pgsql-bugs@lists.postgresql.org
Subject: Re: pg_rewind fails on Windows where tablespaces are used
Date: Mon, 28 Oct 2024 15:00:12 -0400
Message-ID: <982f5caf-6536-4e9f-aeec-227f31114526@dunslane.net> (raw)
In-Reply-To: <ZxmAxLkKrDxc_0zx@paquier.xyz>
References: <CA+pA0kLc5VxOaO4WfLjmh7W0V+quVvVtT5CaRVRAZMuh0zft4Q@mail.gmail.com>
	<Zl6dsK5IcNGTL0qD@paquier.xyz>
	<5169f6d5-240e-49ec-9265-ddafa6e863bd@dunslane.net>
	<Zo3D3Y24Kd1eTV8n@paquier.xyz>
	<CAK98qZ0PLMWRKe=_oHjxwKKDQHZ0wb5ntjiefWpe9V1r8J0b6w@mail.gmail.com>
	<ZxmAxLkKrDxc_0zx@paquier.xyz>


On 2024-10-23 We 7:03 PM, Michael Paquier wrote:
> On Wed, Oct 23, 2024 at 11:19:14AM -0500, Alexandra Wang wrote:
>> I encountered this issue while working on the fix for branch 14 and
>> running the tablespace regress test. This simple test is not covered
>> in branch 15’s regress tests, as we started setting
>> allow_in_place_tablespaces = true since commit d6d317db.
> Yes, for the reasons stated in this commit because we rely on
> everything to be on the same host, and tablespace paths would overlap
> across the primary and its replica.
>
>> I also had to backpatch additional commits for branches 12 to 14, as
>> follows:
>>
>> branch 14: e2f0f8ed, af9e6331, and the commits for branch 15
>> [f357233c, c5cb8f3b, 387803d8, and 5fc88c5d53].
>> branches 12 & 13: bed90759, 54fb8c7d, de8feb1f, 101c37cd, and the
>> commits for branch 14.
>>
>> With these additional commits for branches 12 to 14, I’m not sure if
>> it’s worth backpatching, or should we backpatch only to branch 15?
> 12 is going to be EOL in a couple of days, so I'd rather leave it out.
> If it were down to me, I'd also leave 13 and 14 as well, based on
> e2f0f8ed25 to let the beast sleep there.  Perhaps others have a
> different opinion. though.


Well, it seems like it's clearly a bug. I'm never happy leaving bugs 
unfixed. As for 12, what's the point of putting out one last release if 
it's not to fix bugs?


EDB's customer will probably be happy if we just fix 15, but I would 
rather take a broader view and fix it for other possible users too.


I'm traveling for a few days but it was my intention to work on these 
when I am back.


cheers


andrew


--
Andrew Dunstan
EDB: https://www.enterprisedb.com






view thread (18+ messages)  latest in thread

Message-ID: <982f5caf-6536-4e9f-aeec-227f31114526@dunslane.net>
Permalink:  ../982f5caf-6536-4e9f-aeec-227f31114526@dunslane.net/
Also on:    postgresql.org/message-id/982f5caf-6536-4e9f-aeec-227f31114526@dunslane.net

 · 

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-bugs@postgresql.org
  Cc: andrew@dunslane.net, michael@paquier.xyz, alexandra.wang.oss@gmail.com, chris.travers@stormatics.tech, farooq.rashed@desc.gov.ae, pgsql-bugs@lists.postgresql.org
  Subject: Re: pg_rewind fails on Windows where tablespaces are used
  In-Reply-To: <982f5caf-6536-4e9f-aeec-227f31114526@dunslane.net>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox