Return-Path: Delivered-To: apmail-apr-dev-archive@apr.apache.org Received: (qmail 88887 invoked by uid 500); 31 Jan 2003 15:11:46 -0000 Mailing-List: contact dev-help@apr.apache.org; run by ezmlm Precedence: bulk List-Post: List-Help: List-Unsubscribe: List-Subscribe: Delivered-To: mailing list dev@apr.apache.org Received: (qmail 88597 invoked from network); 31 Jan 2003 15:11:43 -0000 Message-ID: <3E3A9226.40702@wstoddard.com> Date: Fri, 31 Jan 2003 10:11:34 -0500 From: Bill Stoddard User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.2.1) Gecko/20021130 X-Accept-Language: en-us, en MIME-Version: 1.0 To: dev@httpd.apache.org, dev@apr.apache.org Subject: Re: Altogether Broken OtherChild logic References: <5.2.0.9.2.20030130144909.02c4edf0@pop3.rowe-clan.net> <5.2.0.9.2.20030124110322.03045588@pop3.rowe-clan.net> <5.2.0.9.2.20030124110322.03045588@pop3.rowe-clan.net> <5.2.0.9.2.20030130144909.02c4edf0@pop3.rowe-clan.net> <5.2.0.9.2.20030130171012.03bc6b10@pop3.rowe-clan.net> In-Reply-To: <5.2.0.9.2.20030130171012.03bc6b10@pop3.rowe-clan.net> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit X-Spam-Rating: daedalus.apache.org 1.6.2 0/1000/N So I 've actually spent some time looking at this... >apr_proc_other_child_check() on Unix came first, afaict. > Right you are. >Now we're left with ery simple problem. OC works on Unix today, and >it's broken on Win32. Unix's logic is well exercised by a larger group, >WinNT's by a much smaller minority. I trust their logic, but the names >don't match up. > >I'm just going to try to wire it all back together such that Unix (working) >stays working, and Win32 is fixed for 2.0.45 within the next few days. > >Yes - it's bogus they called a fn apr_proc_other_child_check() as >*THE* restart signal. But renaming that fn now is probably safer. > I agree. >As for your question about polling, if we cycle every second we waste >cpu - if we sample every few seconds we lose more log entries etc. >If we receive alerts when the otherchild processes die we can react >immediately without the extra loops. > >Bill > In principle I agree but I am not sure the extra complexity of your proposed solution is worthwhile for implementing reliable piped logs. I really hate complex solutions to simple problems. Complexity makes the code more difficult to debug and maintain and raises the entry barrier for new folks interested in joining the project. I often hear the argument for a complex solution in favor of a simpler solution because the complex solution "might be useful for other applications" or is "more extensible", etc. This is a good line of argument and is quite often true, but not always. It -is- possible to over engineer (biggie size :-) software. I'll happily review whatever you come up with, so party on dude. Bill