[sheepdog] [PATCH] sheep : add new strage type "split"

[Saeki Masaki]

On 2014/12/23 21:10, Liu Yuan wrote:
> On Mon, Dec 22, 2014 at 05:22:05PM +0900, Saeki Masaki wrote:
>> This patch is an experimental. So Please check with caution.
>>
>> Current sheepdog stores whole objects in single directory like "/var/lib/sheepdog/obj"
>> This mechanism is difficult to handle massive files when increasing cluster volume.
>>
>> In particular, inode object having special informations about VDI,
>> so it is preferable to divide directory.
>>
>> New storage type is named "split"
>> It separates the inode object and data object to some directory.
>>
>> How to use.
>> specify --store option at the time dog cluster format
>>
>> dog cluster format --store split
>
> I don't have time to review the code but I'd like to give you some advices:
>

Thank you for advice.

> - The patch aims to bring hierachy to the store. How about the name 'tree_store'
>    which implies a tree structure and use "tree" as the driver name

It's good.

> - You should place the new store code into a standalone file, named
>    "xxx_store.c".

It is correct point out. I'll try that way.
However, it seems not simple because md.c and plain_sotre.c
using the function each other.

Regards,
Saeki.

>
> Thanks
> Yuan
>