Hello,

I haven't checked the merged version, but if I spotted correctly from
the patch, it moves the redis response structure in shared memory.

Is is right? If yes, why? Having it in shared memory will create races
between kamailio processes that store results on same container id --
one worker can overwrite on data stored by another ine.

Cheers,
Daniel

On 04.04.17 12:55, Carsten Bock wrote:
> Module: kamailio
> Branch: master
> Commit: 6d87794cac3f6f3bb967d6e0bf6714a4d78f1bd5
> URL: 
> https://github.com/kamailio/kamailio/commit/6d87794cac3f6f3bb967d6e0bf6714a4d78f1bd5
>
> Author: Carsten Bock <cars...@ng-voice.com>
> Committer: Carsten Bock <cars...@ng-voice.com>
> Date: 2017-04-04T12:55:26+02:00
>
> ndb_redis: Added REDIS-Cluster support [BETA]
>
> ---
>
> Modified: src/modules/ndb_redis/doc/ndb_redis.xml
> Modified: src/modules/ndb_redis/doc/ndb_redis_admin.xml
> Modified: src/modules/ndb_redis/ndb_redis_mod.c
> Modified: src/modules/ndb_redis/redis_client.c
> Modified: src/modules/ndb_redis/redis_client.h
>
> ---
>
> Diff:  
> https://github.com/kamailio/kamailio/commit/6d87794cac3f6f3bb967d6e0bf6714a4d78f1bd5.diff
> Patch: 
> https://github.com/kamailio/kamailio/commit/6d87794cac3f6f3bb967d6e0bf6714a4d78f1bd5.patch
>
>
> _______________________________________________
> sr-dev mailing list
> sr-dev@lists.sip-router.org
> http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev

-- 
Daniel-Constantin Mierla
www.twitter.com/miconda -- www.linkedin.com/in/miconda
Kamailio Advanced Training - May 22-24 (USA) - www.asipto.com
Kamailio World Conference - May 8-10, 2017 - www.kamailioworld.com


_______________________________________________
sr-dev mailing list
sr-dev@lists.sip-router.org
http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev

Reply via email to