[Date Prev][Date Next] [Chronological] [Thread] [Top]

Re: commit: ldap/servers/slapd/back-bdb back-bdb.h config.c tools.c

Hallvard B Furuseth wrote:

Howard Chu writes:

Log Message:
Added linearindex option for slapindex. Improves performance when database
size exceeds dbcache size by minimizing cache thrashing.

Another note that I forgot to mention - with the linearindex option, slapadd does no indexing at all. (Otherwise it would defeat the purpose of this feature.) You must run slapindex after slapadd in this case. I suppose this should be in the manpage too.

Wouldn't it be better to have slapadd run slapindex before it exits?
It might not even have to exec() slapindex, since they are the same
executable now.

Maybe. That might require some other restructuring in the slapadd/slapindex frontend. Currently the patch is isolated to the back-bdb backend, and it could be messy to expose this change in the frontend. I haven't really investigated what it would take, and don't have the time to look into it at the moment.
-- Howard Chu
Chief Architect, Symas Corp. Director, Highland Sun
http://www.symas.com http://highlandsun.com/hyc
Symas: Premier OpenSource Development and Support