rack-devel archive mirror (unofficial) https://groups.google.com/group/rack-devel
 help / color / mirror / Atom feed
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 --]

  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).