On 01/24/2014 01:37 PM, MauMau wrote:
> Hi, Heiki-san,
>
> From: "MauMau" <maumau307@gmail.com>
>> From: "Heikki Linnakangas" <hlinnakangas@vmware.com>
>>> After some refactoring and fixing bugs in the existing code, I came up
>>> with the attached patch. I called the option simply "recovery_target",
>>> with the only allowed value of "immediate". IOW, if you want to stop
>>> recovery as early as possible, you add recovery_target='immediate' to
>>> recovery.conf. Now that we have four different options to set the
>>> recovery target with, I rearranged the docs slightly. How does this look
>>> to you?
>>
>> I'm almost comfortable with your patch. There are two comments:
>>
>> C1. The following parts seem to be mistakenly taken from my patch. These
>> are not necessary for your patch, aren't they?
>
> I'm going to add the attached new revision of the patch soon, which is
> almost based on yours. All what I modified is removal of parts I mentioned
> above. I confirmed that the original problem could be solved. Thanks.
Thanks, committed!
- Heikki