Re: archive recovery fetching wrong segments - Mailing list pgsql-hackers

From David Steele
Subject Re: archive recovery fetching wrong segments
Date
Msg-id 0b696059-bd4f-8c4f-48f3-f8a65d27439f@pgmasters.net
Whole thread Raw
In response to Re: archive recovery fetching wrong segments  (Grigory Smolkin <g.smolkin@postgrespro.ru>)
Responses Re: archive recovery fetching wrong segments  (Grigory Smolkin <g.smolkin@postgrespro.ru>)
List pgsql-hackers
On 4/6/20 3:23 PM, Grigory Smolkin wrote:
> 
> On 4/6/20 9:17 PM, David Steele wrote:
>> Hi Grigory,
> 
> Hello!
>>
>> On 4/5/20 8:02 PM, Grigory Smolkin wrote:
>>> Hello, hackers!
>>>
>>> I`m investigating a complains from our clients about archive recovery 
>>> speed been very slow, and I`ve noticed a really strange and, I think, 
>>> a very dangerous recovery behavior.
>>>
>>> When running multi-timeline archive recovery, for every requested 
>>> segno startup process iterates through every timeline in restore 
>>> target timeline history, starting from highest timeline and ending in 
>>> current, and tries to fetch the segno in question from this timeline.
>>
>> <snip>
>>
>>> Is there a reason behind this behavior?
>>>
>>> Also I`ve  attached a patch, which fixed this issue for me, but I`m 
>>> not sure, that chosen approach is sound and didn`t break something.
>>
>> This sure looks like [1] which has a completed patch nearly ready to 
>> commit. Can you confirm and see if the proposed patch looks good?
> 
> Well I`ve been testing it all day and so far nothing is broken.

Perhaps I wasn't clear. There is a patch in this thread:


https://www.postgresql.org/message-id/flat/792ea085-95c4-bca0-ae82-47fdc80e146d%40oss.nttdata.com#800f005e01af6cb3bfcd70c53007a2db

which seems to address the same issue and is ready to be committed.

I'd suggest you have a look at that patch and see if it fixes your issue.

Regards,
-- 
-David
david@pgmasters.net



pgsql-hackers by date:

Previous
From: Grigory Smolkin
Date:
Subject: Re: archive recovery fetching wrong segments
Next
From: Tomas Vondra
Date:
Subject: Re: [PATCH] Incremental sort (was: PoC: Partial sort)