A bug found in mmc

3 views
Skip to first unread message

名字太长躲在树后容易被发现:D浪子

unread,
Jul 1, 2009, 9:57:46 PM7/1/09
to memc...@googlegroups.com
Dear sirs:
      I'm using memcache to store millions of key-value pairs.But,when i use the command "stats cachedump slabid" to dump all the key-value ,it works at beginning,but that stops after 50,000 pairs dumped.I looked through the source code (memcache.c,stats.c) ,found this,"unsigned int memlimit = 20 * 1024 * 1024;   /* 2MB max response size */" .So,this causes the result.In order to dump more,i have to assign more space for the buffer ,set memlimit for more.But,this way does not come up to expectations.An ideal way is molloc space dynamically, but i failded to do this .So i write mail for help~ Thanks a lot.

Sincerely yours (memcache user from China)

Dustin

unread,
Jul 1, 2009, 10:39:08 PM7/1/09
to memcached
This mechanism should probably be removed from the server. It's not
recommended anyone use it for anything other than doing a minor sample
on a non-production server. It can't get everything and we won't be
doing work to make it so that it can.

phpman

unread,
Jul 1, 2009, 11:15:29 PM7/1/09
to memcached
I'm going to transform the inner function "do_item_cachedump" to
receive more parameters ,in original code , cachedump receives
3params:slabid,limit,bytes. I'm going to add parameter "offset" for
it ,an to read data start from it. It works,but still in testing and
debugging.

Dustin

unread,
Jul 2, 2009, 12:09:23 AM7/2/09
to memcached
Please don't have any expectation of that going upstream. It will not
be possible to do this in any meaningful way that approaches
correctness or efficiency requirements.

We're really best off removing any trace of this functionality.

dormando

unread,
Jul 5, 2009, 4:28:45 PM7/5/09
to memcached

If you do that, you'll break brian's gearmand memcached queue store
functionality!

and uh. that FUSE client.

In all seriousness we should probably officially mark the thing as
deprecated as of 1.4.0 and remove it in the next major release (not after
a minor though).

Anyone disagree? :)

-Dormando

Reply all
Reply to author
Forward
0 new messages