[illumos-Advocates] [illumos-Developer] WEBREV 755 dmu_recv_stream builds incomplete guid_to_ds_map

Gordon Ross gordon.w.ross at gmail.com
Fri Apr 8 10:56:56 PDT 2011


Approved.  Please send me the "hg export f453f523f699"
or whatever it is after you recommit and update the comment.

Thanks,
Gordon


On Fri, Apr 8, 2011 at 12:54 PM, Alexander Stetsenko <ams at nexenta.com> wrote:
> On 04/08/2011 12:10 AM, Gordon Ross wrote:
>>
>> Might be nice to have a little more detail describing the
>> problem and it's fix in issue 755.
>
> Thanx Gordon
> I`ve updated description with more details. I hope it is now more clear
> there was the bug and what
> the fix do.
>>
>> Otherwise looks good to me.
>>
>> Gordon
>>
>> On Thu, Apr 7, 2011 at 2:48 PM, Alexander Stetsenko<ams at nexenta.com>
>>  wrote:
>>>
>>> This 2 in 1 WEBREV and RTI
>>> It was already reviewed privately by Matthew Ahrens and Garrett D`Amore.
>>> Now it is open to public review.
>>> it was tested as desribed in issue,
>>> also was additional testing with explicit duplicate data in stream.
>>> All worked well.
>>> It also contains RTI required files (attached)
>>>
>>> webrev link:
>>> http://cr.illumos.org/view/rnu8ozl6/
>>>
>>> issue link:
>>> http://www.illumos.org/issues/755
>>>
>>>
>>>
>>> _______________________________________________
>>> Developer mailing list
>>> Developer at lists.illumos.org
>>> http://lists.illumos.org/m/listinfo/developer
>>>
>>>
>
>



More information about the Advocates mailing list