> On Apr 29, 2015, at 10:34 AM, Dbz Fan <
manass...@gmail.com> wrote:
>
> Yup rechecked it, all the 3 sentinels are showing increase in memory . So my configuration is this : 3 pcs A,B,C . A has master and sentinel ; B has slave and sentinel ; C has slave and sentinel. Is the increase possible due to configuring of master/slave on same PC as the sentinel ?
Not likely but it is possible you’re looking at the wrong process as depending on the command line used to launch sentinel and redis they’ll both show as “redis-server’ in the process listing. ;) It is also possible your sentinels aren’t getting a valid connection to your Redis master instances - which would trigger the bug referenced.
When you connect to the sentinel(s) and execute a “sentinel slaves <name>” do the slaves show up? If not, you aren’t establishing an effective connection to the master.
On a side note, it isn’t a good idea to run sentinel on the same nodes as your Redis instances.
> Also in the issue reported here
https://github.com/antirez/redis/issues/2535 suggests to me that the increase was seen in sentinel memory even though memory leak was in hiredis.
> In that context I was thinking maybe the issue could be in jedis .Please correct me here if I am missing anything here.
Jedis is a client library, not a server-side one. If Jedis leaks your client application will grow in memory, not sentinel. The leak described was when Sentinel, using Hiredis, connected to a Redis *server* and could not obtain a valid connection such as bad password, timeout, nod not listening, etc.. In this case Sentinel is acting as a client, so a leak in a client library caused the memory increase to be visible in Sentinel-as-a-client. Similarly if Jedis leaks your application which uses Jedis would see the memory usage, not the server.
It is hypothetically possible that sentinel could have a leak when clients connect and disconnect a lot. Are your clients connecting to Sentinel to learn of the master location at all or merely connecting directly to the master?
Cheers,
Bill