memcached icon indicating copy to clipboard operation
memcached copied to clipboard

Fix truncation of bytes/bytes_read/bytes_written stats

Open aroben opened this issue 10 years ago • 4 comments

These stats are stored in a uint64_t but we were truncating them to 32 bits.

aroben avatar Oct 02 '14 16:10 aroben

hey @aroben , thanks for the PR. We just embed the memcached C client in the gem. So, is this changed in the C client? If so, I can bump it here. Otherwise we might need to send a patch there instead.

arthurnn avatar Feb 06 '16 22:02 arthurnn

So, is this changed in the C client?

Yeah, as mentioned above, newer versions of the C client have already fixed this bug.

aroben avatar Feb 07 '16 22:02 aroben

@aroben , I am looking into update the client here, as it is SUPER old.. but before I have all that done, I need to get CI back to green, and after I have that I will merge this for us.. thanks a lot

arthurnn avatar Feb 08 '16 15:02 arthurnn

The embedded libmemcached has been upgraded to 1.0.18 on master now, which is the newest release, so I think this can be closed now.

dylanahsmith avatar Mar 31 '21 23:03 dylanahsmith