Re: mSHEC pull request (2nd challenge)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

On 30/01/2015 11:28, Miyamae, Takeshi wrote:
> Hi Loic,
> 
> Thank you for your cooperation !
> If we could do something to reduce your burdens, please let us know.

Hopefully tests come out clean, let see :-)

There was originally two files for tests and the one with 364 or something at the end was removed if I'm not mistaken. Is there a reason for that ?

Cheers

> 
> Best regards,
> Takeshi Miyamae
> 
> -----Original Message-----
> From: Loic Dachary [mailto:loic@xxxxxxxxxxx] 
> Sent: Thursday, January 29, 2015 11:38 PM
> To: Miyamae, Takeshi/宮前 剛
> Cc: Ceph Development; Shiozawa, Kensuke/塩沢 賢輔; Nakao, Takanori/中尾 鷹詔; Kawaguchi, Shotaro/川口 翔太朗; Kaga, Yoshihiro/加賀 芳宏; Paul Von-Stamwitz (PVonStamwitz@xxxxxxxxxxxxxx)
> Subject: Re: mSHEC pull request (2nd challenge)
> 
> Hi,
> 
> It looks good and ready for integration testing :-) I just created http://tracker.ceph.com/issues/10686 to track this feature. Since we are late in development cycle and unless someone has a different opinion, I will mark the SHEC plugin as experimental so people know it's new. I will however do that after it is merged in master.
> 
> Cheers
> 
> On 29/01/2015 15:24, Miyamae, Takeshi wrote:
>> Hi Loic,
>>
>> When I reset our commits on our fork repository, the 1st pull request was automatically closed.
>> So, I have issued another pull request of mSHEC.
>>
>> mSHEC : multiple Shingled Erasure Code #3534
>> https://github.com/ceph/ceph/pull/3534
>>
>> As a result, I believe I have squashed our commits into one.
>> I'm grad if you could restart the review.
>>
>> (You already noticed that and gave us labels, thank you!)
>>
>> Best regards,
>> Takeshi Miyamae
>>
> 

-- 
Loïc Dachary, Artisan Logiciel Libre

Attachment: signature.asc
Description: OpenPGP digital signature


[Index of Archives]     [CEPH Users]     [Ceph Large]     [Information on CEPH]     [Linux BTRFS]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux