On Thu, 2008-01-17 at 15:16 -0800, Jiang Qian wrote: > Hi, > My last email was wrong > it actually try to re open in above the > if (!managedConn.isOpen()) { > managedConn.open(route, context, params); > } > > Thanks >
Jiang, What kind of behavior do you expect? Feel free to open an issue in JIRA if you think HttpClient does not do what it is supposed to. I am tied up preparing the release of HttpCore. I'll look into the problem as soon as the release has been taken care of. Cheers Oleg > On Jan 17, 2008 3:00 PM, Jiang Qian <[EMAIL PROTECTED]> wrote: > > Hi, > > Here is what I found: > > in DefaultClientRequestDirector execute method line 348 > > if connection drop by server, it will throw NoHttpResponseException, > > then managedConn.close(); > > and this is allow to retry, so it rerun response = > > requestExec.execute(request, managedConn, context); > > > > goes to HttpRequestExecutor line 201, conn.sendRequestHeader(request); > > > > however, it seems no place to reopen the conn in side sendRequestHeader > > the AbstractHttpClientConnection which implements this method > > is > > public void sendRequestHeader(final HttpRequest request) > > throws HttpException, IOException { > > if (request == null) { > > throw new IllegalArgumentException("HTTP request may not be > > null"); > > } > > assertOpen(); > > this.requestWriter.write(request); > > this.metrics.incrementRequestCount(); > > } > > > > seems no where it does the re-open, > > > > Is it a bug? > > > > -- > > Thanks > > http://www.youtube.com/watch?v=yCM_wQy4YVg > > > > > --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]