[sheepdog] [PATCH] sheep: add simple_store driver again
MORITA Kazutaka
morita.kazutaka at lab.ntt.co.jp
Fri Aug 24 09:15:51 CEST 2012
At Fri, 24 Aug 2012 15:04:56 +0800,
Liu Yuan wrote:
>
> On 08/24/2012 01:37 PM, MORITA Kazutaka wrote:
> > It is a good idea to refine the farm backend store too, but simple
> > store should be added as another driver, I think. The simple driver
> > would become a good tutorial to introduce new drivers.
>
> Okay, I'd suggest use default_xxx as prefix instead of simple_store_xxx,
> because 1) more concise 2) won't cause confusion when people read the
> farm code.
The driver name should be 'default', too? I think we shouldn't use a
name 'simple_store' to avoid confusion.
>
> I'll go to update the farm's io path to use defaul_xxx functions, I
> think this simple store implementation can be nicely applied to farm.
The update will change the data layout of farm. How about introducing
'farm2'? I guess farm and farm2 can share all files other than
farm.c.
Thanks,
Kazutaka
More information about the sheepdog
mailing list