On Tue, Aug 5, 2014 at 9:04 PM, Fujii Masao <masao.fujii@gmail.com> wrote:
> On Tue, Jul 29, 2014 at 7:07 PM, <furuyao@pm.nttdata.co.jp> wrote:
>> I have improved the patch by making following changes:
>>
>> 1. Since stream_stop() was redundant, stream_stop() at the time of WAL file closing was deleted.
>>
>> 2. Change the Flash judging timing for the readability of source code.
>> I have changed the Flash judging timing , from the continuous message after receiving to
>> before the feedbackmassege decision of continue statement after execution.
>
> Thanks for the updated version of the patch!
>
> While reviewing the patch, I found that HandleCopyStream() is still
> long and which decreases the readability of the source code.
> So I feel inclined to refactor the HandleCopyStream() more for better
> readability. What about the attached refactoring patch?
Sorry, I forgot to attached the patch in previous email. So attached.
Regards,
--
Fujii Masao