[sr-dev] git:master:6d87794c: ndb_redis: Added REDIS-Cluster support [BETA]

[Carsten Bock]

Hi Daniel,

i've just noticed, that in one line, I tried to free a reply from
shmem instead of pkgmem (fixed). I only put the list of
REDIS-Connections into shmem, so other workers can access (and re-use)
the newly added connections to REDIS.
Note to myself: I should add some locking to the list....

Thanks,
Carsten

2017-04-04 13:11 GMT+02:00 Daniel-Constantin Mierla <miconda at gmail.com>:
> 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 <carsten at ng-voice.com>
>> Committer: Carsten Bock <carsten at 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 at 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
>



-- 
Carsten Bock
CEO (Geschäftsführer)

ng-voice GmbH
Millerntorplatz 1
20359 Hamburg / Germany

http://www.ng-voice.com
mailto:carsten at ng-voice.com

Office +49 40 5247593-40
Fax +49 40 5247593-99

Sitz der Gesellschaft: Hamburg
Registergericht: Amtsgericht Hamburg, HRB 120189
Geschäftsführer: Carsten Bock
Ust-ID: DE279344284

Hier finden Sie unsere handelsrechtlichen Pflichtangaben:
http://www.ng-voice.com/imprint/