From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <e@80x24.org>
X-Spam-Checker-Version: SpamAssassin 3.3.2 (2011-06-06) on dcvr.yhbt.net
X-Spam-Level: 
X-Spam-ASN: 
X-Spam-Status: No, score=-2.0 required=3.0 tests=ALL_TRUSTED,AWL,BAYES_00,
 URIBL_BLOCKED shortcircuit=no autolearn=unavailable version=3.3.2
X-Original-To: yahns-public@yhbt.net
Received: from localhost (dcvr.yhbt.net [127.0.0.1]) by dcvr.yhbt.net
 (Postfix) with ESMTP id D30D4208AE; Fri, 18 Jul 2014 00:45:47 +0000 (UTC)
Date: Fri, 18 Jul 2014 00:45:47 +0000
From: Eric Wong <e@80x24.org>
To: yahns-public@yhbt.net
Subject: Re: [PATCH] http_client: clear some internal ivars on rack.hijack
Message-ID: <20140718004547.GB12207@dcvr.yhbt.net>
References: <20140718003320.GA14749@dcvr.yhbt.net>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20140718003320.GA14749@dcvr.yhbt.net>
List-Id: <yahns-public@yhbt.net>

Eric Wong <e@80x24.org> wrote:
> We should no longer need HTTP parser or input body upon hijacking.
> Remove references to it so the GC can clean those up.  This relies
> on the Rack application deleting "rack.input" from the Rack env,
> too.

Also, in the future I hope to make the HTTP parser + env thread-local
in the common case like the current read buffer.  This should save
some memory for idle clients and only use more memory with slow
clients.

That's more work, though.