From: Eric Wong <normalperson@yhbt.net>
To: unicorn list <mongrel-unicorn@rubyforge.org>
Cc: Aman Gupta <aman@tmm1.net>
Subject: Re: [PATCH] preload_app can take an optional block for warmup
Date: Sat, 21 Sep 2013 08:49:11 +0000 [thread overview]
Message-ID: <20130921084911.GA29716@dcvr.yhbt.net> (raw)
In-Reply-To: <1379713256-23933-1-git-send-email-aman@tmm1.net>
Aman Gupta <aman@tmm1.net> wrote:
Thanks for the patch!
I expect a commit message body to describe why it is useful.
In particular, what benefit does this have over putting the same
code in config.ru or config/initializer.rb (or similar?)
For user-visible config changes like these, it can be similar/identical
to the added documentation.
Anyways, I agree warming up the app is often necessary, but I'm not
convinced it's necessary change unicorn for it. It makes sense
to warmup apps on servers other than unicorn, too.
> --- a/lib/unicorn/http_server.rb
> +++ b/lib/unicorn/http_server.rb
> @@ -721,6 +721,9 @@ class Unicorn::HttpServer
> Gem.refresh
> end
> self.app = app.call
> + if preload_app.respond_to?(:call)
> + preload_app[app]
> + end
Since you're testing for respond_to?(:call), it should be less confusing
to use "preload_app.call(app)" here instead of "preload_app[app]"
_______________________________________________
Unicorn mailing list - mongrel-unicorn@rubyforge.org
http://rubyforge.org/mailman/listinfo/mongrel-unicorn
Do not quote signatures (like this one) or top post when replying
next prev parent reply other threads:[~2013-09-21 8:49 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-09-20 21:40 [PATCH] preload_app can take an optional block for warmup Aman Gupta
2013-09-21 8:49 ` Eric Wong [this message]
2013-09-21 23:10 ` Aman Gupta
2013-09-23 10:58 ` Eric Wong
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://yhbt.net/unicorn/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20130921084911.GA29716@dcvr.yhbt.net \
--to=normalperson@yhbt.net \
--cc=aman@tmm1.net \
--cc=mongrel-unicorn@rubyforge.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://yhbt.net/unicorn.git/
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).