Re: [asterisk-dev] [Code Review] 4490: astdb: Allow clustering of the Asterisk Database between multiple Asterisk servers

This is definitely the approach that I would prefer as well, as it makes integration with external software much easier. - Brad On Thu, Apr 16, 2015 at 12:37 PM, Jeffrey Ollie wrote: > On Wed, Apr 15, 2015 at 2:11 PM, Mark Michelson > wrote: >> >> On 04/14/2015 12:11 PM, Matthew Jordan wrote: >

Re: [asterisk-dev] [Code Review] 4490: astdb: Allow clustering of the Asterisk Database between multiple Asterisk servers

On Wed, Apr 15, 2015 at 2:11 PM, Mark Michelson wrote: > On 04/14/2015 12:11 PM, Matthew Jordan wrote: > >> >> The question is: is this change worth having, or should it be scrapped >> in favour of some alternate approach that makes use of other >> technology? My feelings won't be hurt if the ans

Re: [asterisk-dev] [Code Review] 4490: astdb: Allow clustering of the Asterisk Database between multiple Asterisk servers

On 04/14/2015 12:11 PM, Matthew Jordan wrote: Yup. SO! The question is: is this change worth having, or should it be scrapped in favour of some alternate approach that makes use of other technology? My feelings won't be hurt if the answer is "ditch it and do something else" - this was a fun pi

Re: [asterisk-dev] [Code Review] 4490: astdb: Allow clustering of the Asterisk Database between multiple Asterisk servers

On Wed, Apr 8, 2015 at 1:03 PM, Mark Michelson wrote: Before I repost the AstDB Clustering review from Review Board onto Gerrit, I wanted to address Mark's concerns below. Depending on what people think, it may be worth just shelving the idea. > > The only things I can think of that are issues

Re: [asterisk-dev] [Code Review] 4490: astdb: Allow clustering of the Asterisk Database between multiple Asterisk servers

--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/4490/#review15135 --- The only things I can think of that are issues here are based o

Re: [asterisk-dev] [Code Review] 4490: astdb: Allow clustering of the Asterisk Database between multiple Asterisk servers

--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/4490/ --- (Updated April 5, 2015, 9:22 p.m.) Review request for Asterisk Developers.

Re: [asterisk-dev] [Code Review] 4490: astdb: Allow clustering of the Asterisk Database between multiple Asterisk servers

> On April 2, 2015, 2:41 p.m., Mark Michelson wrote: > > /trunk/main/db.c, lines 958-977 > > > > > > The dialplan function allows an omitted type to automatically be > > interpreted as "global" but the CLI command

Re: [asterisk-dev] [Code Review] 4490: astdb: Allow clustering of the Asterisk Database between multiple Asterisk servers

--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/4490/#review15032 --- /trunk/funcs/func_db.c

Re: [asterisk-dev] [Code Review] 4490: astdb: Allow clustering of the Asterisk Database between multiple Asterisk servers

--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/4490/ --- (Updated March 29, 2015, 8:46 p.m.) Review request for Asterisk Developers

Re: [asterisk-dev] [Code Review] 4490: astdb: Allow clustering of the Asterisk Database between multiple Asterisk servers

> On March 25, 2015, 5:26 p.m., rmudgett wrote: > > /trunk/main/db.c, lines 493-497 > > > > > > Is clone ref leaked here? Yup. I had initially used RAII_VAR here, and didn't complete the removal. Thanks for catchi

Re: [asterisk-dev] [Code Review] 4490: astdb: Allow clustering of the Asterisk Database between multiple Asterisk servers

--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/4490/#review14836 --- /trunk/funcs/func_db.c

Re: [asterisk-dev] [Code Review] 4490: astdb: Allow clustering of the Asterisk Database between multiple Asterisk servers

--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/4490/ --- (Updated March 25, 2015, 10:35 a.m.) Review request for Asterisk Developer

[asterisk-dev] [Code Review] 4490: astdb: Allow clustering of the Asterisk Database between multiple Asterisk servers

--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/4490/ --- Review request for Asterisk Developers. Repository: Asterisk Description