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

Charles Chance charles.chance at sipcentric.com
Thu Nov 13 23:09:02 CET 2014


Hi Andrey,

I have now committed the DMQ updates to master branch.

Regards,
Charles


On 13 November 2014 09:56, Charles Chance <charles.chance at sipcentric.com>
wrote:

> 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
>>>
>>>
>>


-- 
*Charles Chance*
Managing Director

t. 0121 285 4400    m. 07932 063 891

-- 
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/0a247381/attachment.html>


More information about the sr-dev mailing list