Re: [PATCHES] Infrastructure changes for recovery (v8)

On Tue, Dec 23, 2008 at 5:18 AM, Heikki Linnakangas
<heikki(dot)linnakangas(at)enterprisedb(dot)com> wrote:
> Simon Riggs wrote:
>>
>> On Wed, 2008-12-17 at 23:32 -0300, Alvaro Herrera wrote:
>>>
>>> Simon Riggs escribió:
>>>
>>>> Please let me know how I can make the reviewer's job easier. Diagrams,
>>>> writeups, whatever. Thanks,
>>>
>>> A link perhaps?
>>
>> There is much confusion on this point for which I'm very sorry.
>>
>> I originally wrote "infra" patch to allow it to be committed separately
>> in the Sept commitfest, to reduce size of the forthcoming hotstandby
>> patch. That didn't happen (no moans there) so the eventual "hotstandby"
>> patch includes all of what was the infra patch, plus the new code.
>>
>> So currently there is no separate "infra" patch. The two line items on
>> the CommitFest page are really just one large project. I would be in
>> favour of removing the "infra" lines from the CommitFest page.
>
> I think it's useful to review the "infra" part of the patch separately, so I
> split it out of the big patch again. I haven't looked at this in detail yet,
> but it compiles and passes regression tests.

Super! I would fix synch rep code based on this patch.

Regards,

--
Fujii Masao
NIPPON TELEGRAPH AND TELEPHONE CORPORATION
NTT Open Source Software Center