mirror of mongrel-development@rubyforge.org (inactive)
 help / color / mirror / Atom feed
From: Evan Weaver <evan-72XWLPH10WVXUHR/Jj/Uug@public.gmane.org>
To: mongrel-development-GrnCvJ7WPxnNLxjTenLetw@public.gmane.org
Subject: Re: cornifying Mongrel?
Date: Thu, 9 Apr 2009 17:34:15 -0400	[thread overview]
Message-ID: <b6f68fc60904091434m2add2298j3e3b8746ecc71b1e@mail.gmail.com> (raw)
In-Reply-To: <20090409020925.GA26434-yBiyF41qdooeIZ0/mPfg9Q@public.gmane.org>

Ah, awesome.

Next weekend is my first free weekend in a month and I will attempt to
work on it then.


On Wed, Apr 8, 2009 at 10:09 PM, Eric Wong <normalperson-rMlxZR9MS24@public.gmane.org> wrote:
> Hi all,
> Unicorn development has has been pretty boring lately, so maybe it's
> time to merge some of the improvements back into Mongrel2.  I think most
> of the Unicorn changes can be merged into Mongrel with very few
> modifications.  I'll give a highish-level rundown of what I changed from
> Mongrel (since 9f9a9d488ed32a2891dc3dd7d50a17a16357042d)
> I'll start backwards with HttpResponse since that's the least
> intrusive.
> == HttpResponse
> I removed the old Mongrel response interface entirely and just made it
> work directly with the Rack response tuplet.  The most important change
> to me was to stop slurping the entire body into one StringIO.
> I've also added support for Rack-style multi-value headers to
> allow duplicate headers joined by "\n" in the value.  I consider
> suppressing duplicates for certain headers to be the job of Rack
> or the framework, so Unicorn itself does no duplicate suppression.
> Additionally, Content-Length: calculation is gone as Rack or the
> framework will do that (or send "Transfer-Encoding: chunked").
> Generally, these changes avoid doing work Rack has already done for us.
> == HttpParser
> I've simplified interface for HttpRequest a bit.  It relies solely on
> exceptions for errors so callers won't have to explicitly check for
> them.  There's no longer a need to do nread accounting in the caller
> since it's always been done internally.  There are also some small
> bugfixes for some Rack-isms in there.
> == HttpRequest
> Some of these are tied to the HttpParser changes.  The HttpParser
> simplification allows me to avoid some function calls for some small
> performance boosts.
> I've also renamed "HTTP_BODY" in the HttpParser to :http_body to avoid
> conflicting with a theoretical "Body:" HTTP header.
> == Listener
> Unicorn doesn't override TCPListener.new. It supports multiple listeners
> within the same process and also allows fine-grained control of backlog,
> sndbuf, rcvbuf on a per-listener basis.  Lowering listen backlog size
> actually has better potential for failover, but it's unlikely I'll get
> to test that soon on a real production cluster.  Of course, it can
> bind to UNIX domain sockets as well as TCP ones.
> I'll probably document the listener inheritance between upgrades feature
> separately since I don't think I've ever seen it documented (or used
> outside of nginx; let me know if it has).  I'm not sure if Mongrel can
> support it portably outside of POSIX, probably not...
> == Misc
> The global log-reopening facility in Unicorn will be less clean in the
> presence of threads.  Unicorn waits until the current request is
> completely finished before reopening the logs; this allows requests that
> generate multi-line logs to keep all logs within the same file for
> easier processing.  Nevertheless, Unicorn::Util.reopen_logs would
> be preferable over logrotate doing copytruncate.
> == General
> Throughout Unicorn, I use sysread/syswrite instead of the more common
> read/write functions.  I personally find luserspace I/O-buffering
> distasteful, especially on sockets.  These calls should be safely
> switchable to use read/write for Mongrel since it has threads
> and slow clients to deal with (neither are a concern for Unicorn).
> The process management is generally boring and generic (besides reexec).
> It can probably be used in a future mongrel_cluster and allowed shared
> listeners, too.  With preload_app, it works out-of-the box for nearly
> everything (Sinatra 0.3.x with code reloading enabled being an
> exception).
> Tests can run in parallel using GNU make (my favorite language for
> parallel programming).  This was one of the first things I did to the
> original Mongrel code (though I tweaked it along the way).
> More details for all of these changes are of course in the commit
> messages and comments, even.  Unicorn is probably the most heavily
> commented/documented code I've ever written, too.
> Anyways, let me know if you guys want more details on certain
> changes or help merging certain changes into Mongrel.
> == Disclaimer
> Keep in mind that Unicorn still has no real Ruby applications running on
> it (fork+exec on cgit doesn't count).  The Sinatra projects I initially
> tested this on both got canceled, even, so Unicorn could even be bad
> luck :)
> Since Unicorn is/was a new project, I took many more liberties and risks
> with it than I would with an established project like Mongrel, so keep
> that in mind, too :)
> I also don't think I'll be allowed much more $DAYJOB time to work on it.
> Most of our projects already run well enough with the nasty
> qrp+num_processors=1 hack I did last year, and these projects have been
> in code/feature freeze for some time, so I don't think I'll ever be
> allowed to deploy Unicorn with them... Oh well.
> --
> Eric Wong
> _______________________________________________
> Mongrel-development mailing list
> Mongrel-development-GrnCvJ7WPxnNLxjTenLetw@public.gmane.org
> http://rubyforge.org/mailman/listinfo/mongrel-development

Evan Weaver

      parent reply	other threads:[~2009-04-09 21:48 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-04-09  2:09 cornifying Mongrel? Eric Wong
     [not found] ` <20090409020925.GA26434-yBiyF41qdooeIZ0/mPfg9Q@public.gmane.org>
2009-04-09 21:34   ` Evan Weaver [this message]
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).