twitter / twemproxy

A fast, light-weight proxy for memcached and redis
Apache License 2.0
12.15k stars 2.06k forks source link

possible bug in slots count reported in verbose logging statement #673

Open skygragon opened 2 years ago

skygragon commented 2 years ago

why add continuum_addition again? seems nserver_continuum already take it in count

image

TysonAndre commented 2 years ago

Yeah, it looks like continuum_addition (*_CONTINUUM_ADDITION) was already added, so the logged number of slots allocated (e.g. with erealloc) is wrong