Re: Crash on promotion when recovery.conf is renamed

> On 02 Sep 2017, at 14:17, Michael Paquier <michael(dot)paquier(at)gmail(dot)com> wrote:
>
> On Fri, Sep 1, 2017 at 7:17 PM, Thomas Munro
> <thomas(dot)munro(at)enterprisedb(dot)com> wrote:
>> On Sat, Aug 26, 2017 at 7:32 AM, Robert Haas <robertmhaas(at)gmail(dot)com> wrote:
>>> On Sat, Apr 8, 2017 at 10:05 AM, David Steele <david(at)pgmasters(dot)net> wrote:
>>>> On 3/28/17 1:21 AM, Tsunakawa, Takayuki wrote:
>>>>> Thanks, marked as ready for committer, as the code is good and Alexander reported the test success.
>>>>
>>>> This bug has been moved to CF 2017-07.
>>>
>>> This bug fix has been pending in "Ready for Committer" state for about
>>> 4.5 months. Three committers (Magnus, Heikki, Tom) have contributed
>>> to the thread to date. Maybe one of them would like to commit this?
>>
>> In the meantime its bits have begun to rot. Michael, could you please rebase?
>
> Thanks for the reminder, Thomas. The 2PC code triggered during
> recovery has been largely reworked in PG10, explaining the conflicts.
> As this has been some time since I touched this patch, I had again a
> look at its logic and could not find any problems around it. So
> attached are a rebased versions for both 0001 and 0002, I have updated
> the messages as well to be more in-line with what is in HEAD for
> corrupted entries.

I’ve moved this to the next CF, but since this no longer applies cleanly I’ve
reset it to Waiting for author.

cheers ./daniel