From: Yehuda Katz <wycats@gmail.com>
To: rack-devel@googlegroups.com
Subject: Re: Bloat?
Date: Thu, 13 Aug 2009 10:17:59 -0700 [thread overview]
Message-ID: <4A844AC7.1020908@gmail.com> (raw)
In-Reply-To: <2a8d4a710908131004i2cef7cb7lc0be17ae7fe7619f@mail.gmail.com>
[-- Attachment #1: Type: text/plain, Size: 1747 bytes --]
I think Rack::Request and Rack::Response are convenient, but they need
to undergo a fairly extensive performance audit if they are to remain in
rack-core.
You might have seen that I did a bunch of performance work on Rails 3,
and I got a lot of improvement by moving pieces of Rack::Response down
into ActionDispatch::Response and rewriting them for performance. For
instance, if people want a HeaderHash, they should probably have to
specify that. Right now, there's no way to super to
Rack::Response#initialize and avoid creating a fairly expensive HeaderHash.
My approach would be that Rack::Response would be extremely simple and
performant, and that if people wanted to add special functionality to
it, they should do it themselves (and possibly call super). I'd be happy
to supply patches if people were interested, but I'm not sure if
everything could be 100% compatible with the existing Response. Which
frameworks are using Rack::Response?
-- Yehdua
Matt Todd wrote:
> I was talking to a friend of mine yesterday and he mentioned that
> thought the Rack package itself seemed to be slightly bloated by
> things like Basic Auth et al. He mentioned two things I thought were
> interesting and I wanted to get your feedback on it:
>
> 1. Like Merb, Rack probably could benefit from using a core and more
> separation of functionality, and
> 2. Rack core should only include what's necessary for Rack::Lint to
> validate a basic application at minimum.
>
> Thoughts?
>
> Matt
>
> --
> Matt Todd
> Highgroove Studios
> www.highgroove.com <http://www.highgroove.com>
> cell: 404-314-2612
> blog: maraby.org <http://maraby.org>
>
> Scout - Web Monitoring and Reporting Software
> www.scoutapp.com <http://www.scoutapp.com>
[-- Attachment #2: Type: text/html, Size: 2318 bytes --]
next prev parent reply other threads:[~2009-08-13 17:18 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-08-13 17:04 Bloat? Matt Todd
2009-08-13 17:17 ` Yehuda Katz [this message]
2009-08-13 17:45 ` Bloat? James Tucker
2009-08-13 17:49 ` Bloat? Joshua Peek
2009-08-13 17:50 ` Bloat? James Tucker
2009-08-13 17:55 ` Bloat? Adrian Madrid
2009-08-13 18:54 ` Bloat? James Tucker
2009-08-13 20:20 ` Bloat? Christian Neukirchen
2009-08-13 17:49 ` Bloat? James Tucker
2009-08-13 19:01 ` Bloat? Ryan Tomayko
2009-08-13 19:19 ` Bloat? Matt Todd
2009-08-13 22:26 ` Bloat? James Tucker
2009-08-15 3:30 ` Bloat? Kyle Drake
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-list from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://groups.google.com/group/rack-devel
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4A844AC7.1020908@gmail.com \
--to=rack-devel@googlegroups.com \
/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.
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).