hc-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jiang Qian" <julianq...@google.com>
Subject Re: 4.0 does not retry when connect drop?
Date Fri, 18 Jan 2008 00:15:39 GMT
Hi Oleg
  thanks!
  I am debugging a NoHttpResponseException issue, and my initial
thoughts was wrong, it is not related to retry.

On Jan 17, 2008 4:10 PM, Oleg Kalnichevski <olegk@apache.org> wrote:
>
> 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 <julianqian@google.com> 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: dev-unsubscribe@hc.apache.org
> For additional commands, e-mail: dev-help@hc.apache.org
>
>



-- 
Thanks
http://www.youtube.com/watch?v=yCM_wQy4YVg

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


Mime
View raw message