Re: [PATCH v4 20/21] refs: add LMDB refs storage backend

2016-02-17 Thread David Turner
On Sun, 2016-02-14 at 19:04 +0700, Duy Nguyen wrote: > On Sat, Feb 6, 2016 at 2:44 AM, David Turner < > dtur...@twopensource.com> wrote: > > +static char *get_refdb_path(const char *base) > > +{ > > + static struct strbuf path_buf = STRBUF_INIT; > > + strbuf_reset(&path_buf); > > +

Re: [PATCH v4 20/21] refs: add LMDB refs storage backend

2016-02-16 Thread David Turner
On Mon, 2016-02-15 at 16:57 +0700, Duy Nguyen wrote: > On Sun, Feb 14, 2016 at 7:04 PM, Duy Nguyen > wrote: > > On Sat, Feb 6, 2016 at 2:44 AM, David Turner < > > dtur...@twopensource.com> wrote: > > > +static char *get_refdb_path(const char *base) > > > +{ > > > + static struct strbuf path_

Re: [PATCH v4 20/21] refs: add LMDB refs storage backend

2016-02-15 Thread Duy Nguyen
On Sun, Feb 14, 2016 at 7:04 PM, Duy Nguyen wrote: > On Sat, Feb 6, 2016 at 2:44 AM, David Turner wrote: >> +static char *get_refdb_path(const char *base) >> +{ >> + static struct strbuf path_buf = STRBUF_INIT; >> + strbuf_reset(&path_buf); >> + strbuf_addf(&path_buf, "%s/refdb"

Re: [PATCH v4 20/21] refs: add LMDB refs storage backend

2016-02-14 Thread Duy Nguyen
On Sat, Feb 6, 2016 at 2:44 AM, David Turner wrote: > +static char *get_refdb_path(const char *base) > +{ > + static struct strbuf path_buf = STRBUF_INIT; > + strbuf_reset(&path_buf); > + strbuf_addf(&path_buf, "%s/refdb", base); > + return path_buf.buf; > +} ... > +static

Re: [PATCH v4 20/21] refs: add LMDB refs storage backend

2016-02-12 Thread David Turner
On Fri, 2016-02-12 at 18:01 +0100, Michael Haggerty wrote: > On 02/05/2016 08:44 PM, David Turner wrote: > > Add a database backend for refs using LMDB. This backend runs git > > for-each-ref about 30% faster than the files backend with fully > > -packed > > refs on a repo with ~120k refs. It's a

Re: [PATCH v4 20/21] refs: add LMDB refs storage backend

2016-02-12 Thread Michael Haggerty
On 02/05/2016 08:44 PM, David Turner wrote: > Add a database backend for refs using LMDB. This backend runs git > for-each-ref about 30% faster than the files backend with fully-packed > refs on a repo with ~120k refs. It's also about 4x faster than using > fully-unpacked refs. In addition, and

Re: [PATCH v4 20/21] refs: add LMDB refs storage backend

2016-02-11 Thread David Turner
On Thu, 2016-02-11 at 09:48 +0100, Michael Haggerty wrote: > On 02/05/2016 08:44 PM, David Turner wrote: > > Add a database backend for refs using LMDB. This backend runs git > > for-each-ref about 30% faster than the files backend with fully > > -packed > > refs on a repo with ~120k refs. It's a

Re: [PATCH v4 20/21] refs: add LMDB refs storage backend

2016-02-11 Thread Michael Haggerty
On 02/05/2016 08:44 PM, David Turner wrote: > Add a database backend for refs using LMDB. This backend runs git > for-each-ref about 30% faster than the files backend with fully-packed > refs on a repo with ~120k refs. It's also about 4x faster than using > fully-unpacked refs. In addition, and

[PATCH v4 20/21] refs: add LMDB refs storage backend

2016-02-05 Thread David Turner
Add a database backend for refs using LMDB. This backend runs git for-each-ref about 30% faster than the files backend with fully-packed refs on a repo with ~120k refs. It's also about 4x faster than using fully-unpacked refs. In addition, and perhaps more importantly, it avoids case-conflict is