Re: [PATCHv2] Add support for LMDB header cache backend

2017-02-06 Thread Christian Ebert
* Kevin J. McCarthy on Saturday, February 04, 2017 at 13:16:21 -0800 > I've pushed this up, with some additional fixes in a separate patch. > Please let me know if I bungled anything. This gives quite a speed boost over tokyocabinet here. Great. So far works flawlessly for me[tm]. Thanks. -- t

Re: [PATCHv2] Add support for LMDB header cache backend

2017-02-04 Thread Kevin J. McCarthy
On Fri, Jan 27, 2017 at 10:47:38AM +0100, Fabian Groffen wrote: > On 27-01-2017 09:58:52 +0100, Fabian Groffen wrote: > > > Two minor issues: > > > - You may want to set LMDB_DB_SIZE to a larger value, see > > >https://github.com/neomutt/neomutt/issues/267 > > > > Ok, I'll add that. > > > >

Re: [PATCHv2] Add support for LMDB header cache backend

2017-01-27 Thread Clemens Lang
Hi, On Fri, Jan 27, 2017 at 10:47:38AM +0100, Fabian Groffen wrote: > On 27-01-2017 09:58:52 +0100, Fabian Groffen wrote: > > > - There's a missing free(&ctx) in hcache_lmdb_open() when > > >mdb_env_create() fails. > > > > I'll add it, thanks. > > I looked at this but can't see a problem in

[PATCHv2] Add support for LMDB header cache backend

2017-01-27 Thread Fabian Groffen
On 27-01-2017 09:58:52 +0100, Fabian Groffen wrote: > > Two minor issues: > > - You may want to set LMDB_DB_SIZE to a larger value, see > >https://github.com/neomutt/neomutt/issues/267 > > Ok, I'll add that. > > > - There's a missing free(&ctx) in hcache_lmdb_open() when > >mdb_env_crea