[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