|
From: | Jeffrey Johnson |
Subject: | Re: [Sks-devel] [Announcement] SKS 1.1.6 Released |
Date: | Mon, 08 Aug 2016 23:00:29 -0400 |
A couple of nitpicks, peripheral to BDB, specific to SKS, related to pagesize: The sampleConfig/sksconf.typical file includes these lines: # KDB/key 65536 pagesize: 128 # # KDB/keyid 32768 keyid_pagesize: 64 1) The naming of “pagesize” (likely BDB hysterical naming) does not follow the other table(s) naming: one would expect “key_pagesize”. 2) There is no “keyid_pagesize”. In fact, if you set that variable, then sks fails to run with an obscure error. (there’s likely a different config variable name in the sources, haven’t looked). The second failure is seen iff sksconf is in the current directory when running sks_build.sh. Which brings up another another nitpick: if trying to set per-table pagesize using sksconf, then a warning message might be useful if the current directory does *NOT* contain a valid sksconf. (aside: note-to-self) There is likely a way to change pagesize without having to reload from a dump. I’ll see if I can dig out the details. todo++. hth |
[Prev in Thread] | Current Thread | [Next in Thread] |