Finally replying to this now, because somebody just referenced it and I see nobody else has replied.<br><br>Some comments:<br><br>* include your patch inline or post a link to it that's easy to see in a browser. you'll get more feedback.<br>
<br>* a tarball's the wrong format. just post a unified diff (diff -u)<br><br>* we don't about diffs to auto-generated files like configure scripts. just the source.<br><br>* #ifdefs all over source code are ugly<br>
<br>* please follow local style. that is, look at code around you and make your new code look like existing code.<br><br>* magic exptime value? magic values are gross. don't we have flag bits left?<br><br>* what happens when one LRU-locked amidst thousands of non-LRU-locked items gets to the tail of the queue? are we locked forever. do you promote the LRU-locked one to the head?<br>
<br>etc<br><br><br><div class="gmail_quote">On Sun, Mar 25, 2007 at 8:18 PM, Paul G <<a href="mailto:paul-lists@perforge.com">paul-lists@perforge.com</a>> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
folks,<br>
<br>
i've put together a small patch (in fact, most of it is autoconf stuff) to allow writing 'permanent' items which won't be subject to LRU replacement. this is useful (i think) if you want to guarantee that some data survives either until its expiration or until the server/machine it's on dies. while this is obviously doable with the appropriate command line switch, this patch allows it to be specified on a per item basis and changed at any time, meaning you can store them in the same memory pressured memcached instance you're hammering with everything else.<br>
<br>
testing and comments from memcached hackers wrt what i've subtly broken with this would be most welcome. please note, while it works in my tests (such as they are), i *don't* guarantee that it won't eat your children or sleep with your wife ;)<br>
<br>
usage:<br>
apply diff, autoconf + autoheader, then<br>
./configure --enable-perm-items<br>
<br>
(i've tried to make it as simple to try out as possible, admittedly so i can get some testing and comments on it).<br>
<br>
<br>
for other options and a few short notes on the why/what/how, check the included README.<br>
<br>
you can grab it here: <a href="http://www.perforge.com/permitems-patch-memcached-1.2.1.tgz" target="_blank">http://www.perforge.com/permitems-patch-memcached-1.2.1.tgz</a><br>
<br>
cheers,<br><font color="#888888">
-p<br>
<br>
<br>
<br>
</font></blockquote></div><br>