Re: mod_watchdog API, was Re: svn commit: r759751 - /httpd/httpd/trunk/modules/cluster/mod_heartbeat.c

2009-03-30 Thread Mladen Turk
Jim Jagielski wrote: fine, disagree with that, but what are your thoughts on switching to the provider API? +1 to provider... it's one of the best parts of httpd But it uses the provider already :) Regards -- ^(TM)

Re: mod_watchdog API, was Re: svn commit: r759751 - /httpd/httpd/trunk/modules/cluster/mod_heartbeat.c

2009-03-30 Thread Jim Jagielski
On Mar 30, 2009, at 5:22 AM, Paul Querna wrote: On Mon, Mar 30, 2009 at 10:48 AM, Mladen Turk wrote: Paul Querna wrote: On Sun, Mar 29, 2009 at 10:44 PM, Paul Querna wrote: Inside a MPM that does it natively, just use the registered list of providers, and implement the same behavoirs

Re: mod_watchdog API, was Re: svn commit: r759751 - /httpd/httpd/trunk/modules/cluster/mod_heartbeat.c

2009-03-30 Thread Mladen Turk
Paul Querna wrote: On Mon, Mar 30, 2009 at 10:48 AM, Mladen Turk wrote: Paul Querna wrote: On Sun, Mar 29, 2009 at 10:44 PM, Paul Querna wrote: Inside a MPM that does it natively, just use the registered list of providers, and implement the same behavoirs as the module. The problem with mp

Re: mod_watchdog API, was Re: svn commit: r759751 - /httpd/httpd/trunk/modules/cluster/mod_heartbeat.c

2009-03-30 Thread Paul Querna
On Mon, Mar 30, 2009 at 10:48 AM, Mladen Turk wrote: > Paul Querna wrote: >> >> On Sun, Mar 29, 2009 at 10:44 PM, Paul Querna wrote: >> >> Inside a MPM that does it natively, just use the registered list of >> providers, and implement the same behavoirs as the module. >> > > The problem with mpm

Re: mod_watchdog API, was Re: svn commit: r759751 - /httpd/httpd/trunk/modules/cluster/mod_heartbeat.c

2009-03-30 Thread Mladen Turk
Paul Querna wrote: On Sun, Mar 29, 2009 at 10:44 PM, Paul Querna wrote: Inside a MPM that does it natively, just use the registered list of providers, and implement the same behavoirs as the module. The problem with mpm is that (IIRC the proposal) it uses the current free resource. However i

Re: mod_watchdog API, was Re: svn commit: r759751 - /httpd/httpd/trunk/modules/cluster/mod_heartbeat.c

2009-03-29 Thread Paul Querna
On Sun, Mar 29, 2009 at 10:44 PM, Paul Querna wrote: > On Sun, Mar 29, 2009 at 10:25 PM, Mladen Turk wrote: >> Paul Querna wrote: >>> >>> On Sun, Mar 29, 2009 at 9:17 PM,   wrote: Author: mturk Date: Sun Mar 29 19:17:30 2009 New Revision: 759751 URL: http://svn.apach

Re: mod_watchdog API, was Re: svn commit: r759751 - /httpd/httpd/trunk/modules/cluster/mod_heartbeat.c

2009-03-29 Thread Paul Querna
On Sun, Mar 29, 2009 at 10:25 PM, Mladen Turk wrote: > Paul Querna wrote: >> >> On Sun, Mar 29, 2009 at 9:17 PM,   wrote: >>> >>> Author: mturk >>> Date: Sun Mar 29 19:17:30 2009 >>> New Revision: 759751 >>> >>> URL: http://svn.apache.org/viewvc?rev=759751&view=rev >>> Log: >>> Use child singleton

Re: mod_watchdog API, was Re: svn commit: r759751 - /httpd/httpd/trunk/modules/cluster/mod_heartbeat.c

2009-03-29 Thread Mladen Turk
Paul Querna wrote: On Sun, Mar 29, 2009 at 9:17 PM, wrote: Author: mturk Date: Sun Mar 29 19:17:30 2009 New Revision: 759751 URL: http://svn.apache.org/viewvc?rev=759751&view=rev Log: Use child singleton watchdog for running the heartbeat module Modified: httpd/httpd/trunk/modules/cluster

Re: mod_watchdog API, was Re: svn commit: r759751 - /httpd/httpd/trunk/modules/cluster/mod_heartbeat.c

2009-03-29 Thread Graham Leggett
Paul Querna wrote: I'm looking at the API exposed by mod_watchdog.h, and its not quite feeling right. If we want this to be an API used easily, I think we should just make them proper timers (ie, run function X in singleton in 10 seconds), and when the timer finishes, it can re-register -- this

mod_watchdog API, was Re: svn commit: r759751 - /httpd/httpd/trunk/modules/cluster/mod_heartbeat.c

2009-03-29 Thread Paul Querna
On Sun, Mar 29, 2009 at 9:17 PM, wrote: > Author: mturk > Date: Sun Mar 29 19:17:30 2009 > New Revision: 759751 > > URL: http://svn.apache.org/viewvc?rev=759751&view=rev > Log: > Use child singleton watchdog for running the heartbeat module > > Modified: >    httpd/httpd/trunk/modules/cluster/mod