mirror of mongrel-development@rubyforge.org (inactive)
 help / color / mirror / Atom feed
From: Wayne Seguin <wayneeseguin-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
To: mongrel-development-GrnCvJ7WPxnNLxjTenLetw@public.gmane.org
Subject: Re: Mongrel 2.0
Date: Wed, 19 Nov 2008 21:31:49 -0500	[thread overview]
Message-ID: <4FF8586B-5B06-4F63-8A10-531AAC942EB3@gmail.com> (raw)
In-Reply-To: <b6f68fc60811172149i5d0c8261ldba15c02212d6a0-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>

Evan,

I believe that optional with default being to drop the connection  
would be best in this case. It will serve the 98% case just fine and  
for those who are more demanding they can opt in.

The rest look like a good set for a 2.0 release to me.

   ~Wayne

On Nov 18, 2008, at 00:49 , Evan Weaver wrote:
> Optional is a good compromise.
>
> Evan
>
> On Tue, Nov 18, 2008 at 12:36 AM, Zed A. Shaw <zedshaw-dd7LMGGEL7NBDgjK7y7TUQ@public.gmane.org>  
> wrote:
>> Very cool, nice work.  One small comment...
>>
>> On Mon, Nov 17, 2008 at 03:32:33PM -0500, Evan Weaver wrote:
>>> Hi Mongrels,
>>>
>>> - Error code instead of connection-close, as discussed earlier
>>
>> With this do you mean returning one of the many error codes in an  
>> HTTP
>> response when the server is overloaded?
>>
>> I would recommend against that if that's the case.  In theory it's  
>> nice
>> to return an error code to clients that could be waiting.  In  
>> practice,
>> your queue is already overloaded, so taking more time to send a  
>> response
>> to clients only adds to the problem.  In a modern system this isn't  
>> such
>> a big deal, but in Ruby it becomes a mess because of the file id  
>> limits
>> in the select loop.
>>
>> I'd say, if you add that, make it optional to turn it off and be
>> prepared with a FAQ about it in case people have problems.
>>
>> Have fun.
>>
>> --
>> Zed A. Shaw
>> http://freehackersunion.org/
>> _______________________________________________
>> Mongrel-development mailing list
>> Mongrel-development-GrnCvJ7WPxnNLxjTenLetw@public.gmane.org
>> http://rubyforge.org/mailman/listinfo/mongrel-development
>>
>
>
>
> -- 
> Evan Weaver
> _______________________________________________
> Mongrel-development mailing list
> Mongrel-development-GrnCvJ7WPxnNLxjTenLetw@public.gmane.org
> http://rubyforge.org/mailman/listinfo/mongrel-development

  parent reply	other threads:[~2008-11-20  2:33 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-11-17 20:32 Evan Weaver
     [not found] ` <b6f68fc60811171232s46ba20ebh19ff2d499494622d-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2008-11-17 20:33   ` Evan Weaver
     [not found]     ` <b6f68fc60811171233y7f103da0od5e875007f153f15-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2008-11-17 20:47       ` Ezra Zygmuntowicz
     [not found]         ` <B99E038D-E0E5-43D1-8DCA-08957639F2CF-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2008-11-17 22:55           ` Filipe Lautert
2008-11-18  5:36   ` Zed A. Shaw
2008-11-18  5:49     ` Evan Weaver
     [not found]       ` <b6f68fc60811172149i5d0c8261ldba15c02212d6a0-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2008-11-20  2:31         ` Wayne Seguin [this message]
2008-11-18 14:59     ` Kirk Haines
     [not found]       ` <f4cd26df0811180659n218275a4hf06313ca8f48eed0-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2008-11-18 16:22         ` Evan Weaver
2008-11-20  3:09   ` Luis Lavena
     [not found]     ` <71166b3b0811191909n6ad79b3cse89be4b9c50b8283-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2008-11-20  4:39       ` Kirk Haines
     [not found]         ` <f4cd26df0811192039r7b96da46j5fde78edcc80cf91-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2008-11-20  5:54           ` Tony Arcieri
     [not found]             ` <c7e6b2b00811192154i1c2fb8f9g4cf70045c277e2a0-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2008-11-21 18:03               ` Evan Weaver
2008-12-06  0:48   ` Eric Wong
replies disabled, historical list

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).