Mailing-List: contact dev-help@hc.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "HttpComponents Project" <dev@hc.apache.org>
MIME-Version: 1.0
Date: Tue, 19 Jul 2011 11:53:48 -0400
Message-ID: 
 <CAMHjJ=Qa64unQjzoP7c-6OU2p8+cWQM+jnzeDotRr57Efp7UsQ@mail.gmail.com>
Subject: question about ConnPoolByRoute#freeEntry
From: Jon Moore <jonm@apache.org>
To: HttpComponents Project <dev@hc.apache.org>
Content-Type: text/plain; charset=ISO-8859-1

I was in the process of working on the connection reuse strategy in
the context of HTTPCLIENT-1108, and was adding unit tests, when I ran
across this:

If I obtain a BasicPoolEntry and then return it to the pool via
freeEntry() with reusable set to "false", the underlying connection
doesn't actually get closed there, which seems odd to me. Roughly, the
code says:

if (reusable) {
  // add to free pool
} else {
  rospl.dropEntry();  // this doesn't close the entry's connection either
  numConnections--;
}

So it seems like we are just losing all the references to the
underlying OperatedClientConnection; is it getting closed via GC or
finalization then? Seems like adding a closeConnection(entry) to the
"else" clause above would be the right thing to do.

Or am I misunderstanding how this works?

Thanks,
Jon

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@hc.apache.org
For additional commands, e-mail: dev-help@hc.apache.org