[sr-dev] New module DMQ_SYNC and patch for DMQ

Charles Chance charles.chance at sipcentric.com
Thu Nov 13 10:56:37 CET 2014


Hi Andrey,

I will get to it later today.

Cheers,
Charles
 On 13 Nov 2014 09:15, "Andrey Rybkin" <rybkin.a at bks.tv> wrote:

> Hello, Charles!
>
> Can you commit my patch to your DMQ module?
>
> ________________________________
> Andrey Rybkin
> JID: rybkin.a at bks.tv
> E-Mail: rybkin.a at bks.tv
>
> 2014-10-29 21:38 GMT+03:00 Charles Chance <charles.chance at sipcentric.com>:
>
>> Maybe others can offer their advice as to which is the preferred way to
>> go?
>>
>>
>>
>> On 28 October 2014 14:40, Andrey Rybkin <rybkin.a at bks.tv> wrote:
>>
>>> Thought about to add this functionality to the module. If this approach
>>> seems to be more correct, then I can move my code in a module, but, in my
>>> opinion, it is not very correct.
>>> ________________________________
>>> Andrey Rybkin
>>> JID: rybkin.a at bks.tv
>>> E-Mail: rybkin.a at bks.tv
>>>
>>> 2014-10-28 16:46 GMT+03:00 Alex Hermann <alex at speakup.nl>:
>>>
>>>> On Tuesday 28 October 2014, Андрей Рыбкин wrote:
>>>> > Module DMQ_SYNC:
>>>> >   The module add replication between multiple servers via DMQ module.
>>>> >    Currently only usrloc replications.
>>>>
>>>> I think some decision should be made on where syncing code should be
>>>> placed.
>>>> The htable module has dmq sync integrated, now you introduce a module
>>>> dedicated to syncing via dmq.
>>>>
>>>> I have no strong preference, but am against different implementations
>>>> depending on the module. So i would prefer either dmq-sync integrated
>>>> into
>>>> each module, or one dmq_sync module. Not a combination of both.
>>>>
>>>>
>>>> > TODO:
>>>> >  1.  dialog replications.
>>>> >  2.  bugfixes.
>>>> >
>>>> > Patch fo DMQ:
>>>> >  1. add  find_dmq_node_uri function  to API
>>>> >      dmq_node_t* find_dmq_node_uri(str* uri)
>>>>
>>>> >  2. add  init_callback  to  dmq_peer
>>>> >      int init_callback()
>>>>
>>>> Except for the broken indenting, this seems equal to my code. If it is
>>>> mine,
>>>> and the code is useful, please cherry-pick the relevant commits so
>>>> authorship
>>>> is retained.
>>>>
>>>> --
>>>> Greetings,
>>>>
>>>> Alex Hermann
>>>>
>>>>
>>>
>>> _______________________________________________
>>> sr-dev mailing list
>>> sr-dev at lists.sip-router.org
>>> http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev
>>>
>>>
>>
>>
>> www.sipcentric.com
>>
>> Follow us on twitter @sipcentric <http://twitter.com/sipcentric>
>>
>> Sipcentric Ltd. Company registered in England & Wales no. 7365592. Registered
>> office: Faraday Wharf, Innovation Birmingham Campus, Holt Street,
>> Birmingham Science Park, Birmingham B7 4BB.
>> _______________________________________________
>> sr-dev mailing list
>> sr-dev at lists.sip-router.org
>> http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev
>>
>>
>

-- 
www.sipcentric.com

Follow us on twitter @sipcentric <http://twitter.com/sipcentric>

Sipcentric Ltd. Company registered in England & Wales no. 7365592. Registered 
office: Faraday Wharf, Innovation Birmingham Campus, Holt Street, 
Birmingham Science Park, Birmingham B7 4BB.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.sip-router.org/pipermail/sr-dev/attachments/20141113/5dd8657a/attachment.html>


More information about the sr-dev mailing list