hc-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sam Berlin" <sber...@gmail.com>
Subject Re: httpcore regression?
Date Fri, 07 Mar 2008 21:18:34 GMT
Ya, that's basically the reason for the suggestion -- to prevent
adding a bunch of if!=null checks all over.  Good to know that the
Async client handler is being used (if only indirectly) !

Sam

On 3/7/08, Andreas Andreou <andreoua@gmail.com> wrote:
> i'm totally clueless on the internals but this suggestion sounds like
> the null-object pattern and indeed makes sense here (instead of keep finding
> probable null checking situations)
>
> On Fri, Mar 7, 2008 at 11:12 PM, Sam Berlin <sberlin@gmail.com> wrote:
> > I think the change should be to set an empty entity if none is set but
> >  the request is an entity-enclosing one, rather not setting one at all.
> >
> >  Sam
> >
> >
> >
> >  On 3/7/08, Andreas Andreou <andreoua@gmail.com> wrote:
> >  > hmmm - no luck... I now get
> >  >
> >  > Exception in thread "IO Reactor Execution Thread" java.lang.NullPointerException
> >  >        at org.apache.http.nio.protocol.AsyncNHttpClientHandler.outputReady(AsyncNHttpClientHandler.java:222)
> >  >        at org.apache.http.nio.protocol.BufferingHttpClientHandler.outputReady(BufferingHttpClientHandler.java:110)
> >  >        at org.apache.http.impl.nio.DefaultNHttpClientConnection.produceOutput(DefaultNHttpClientConnection.java:170)
> >  >        at org.apache.http.impl.nio.DefaultClientIOEventDispatch.outputReady(DefaultClientIOEventDispatch.java:105)
> >  >
> >  > so (unless the above makes sense for you) perhaps i should try and get
> >  > you a testcase...
> >  >
> >  > On Fri, Mar 7, 2008 at 10:18 PM, Oleg Kalnichevski <olegk@apache.org>
wrote:
> >  > >
> >  > >  On Fri, 2008-03-07 at 00:56 +0200, Andreas Andreou wrote:
> >  > >
> >  > >
> >  > > > Hi list!
> >  > >  > We've been using httpcore & httpcore-nio
> >  > >  > 4.0-beta2-20080229.095821-3 but since the new
> >  > >  > 4.0-beta2-20080303.182830-4 we've been getting (on attempts to
request any url)
> >  > >  > Exception in thread "IO Reactor Execution Thread"
> >  > >  > java.lang.IllegalArgumentException: wrapped entity must not be
null
> >  > >  >         at org.apache.http.entity.HttpEntityWrapper.<init>(HttpEntityWrapper.java:66)
> >  > >  >         at org.apache.http.nio.entity.NHttpEntityWrapper.<init>(NHttpEntityWrapper.java:53)
> >  > >  >         at org.apache.http.nio.protocol.AsyncNHttpClientHandler.requestReady(AsyncNHttpClientHandler.java:156)
> >  > >  >         at org.apache.http.nio.protocol.AsyncNHttpClientHandler.connected(AsyncNHttpClientHandler.java:109)
> >  > >  >         at org.apache.http.nio.protocol.BufferingHttpClientHandler.connected(BufferingHttpClientHandler.java:94)
> >  > >  >         at org.apache.http.impl.nio.DefaultClientIOEventDispatch.connected(DefaultClientIOEventDispatch.java:87)
> >  > >  >         at com.amplafi.core.iomanagement.http.UniversalIOEventDispatch.connected(UniversalIOEventDispatch.java:102)
> >  > >  >         at $IOEventDispatch_1187ff96e6e.connected($IOEventDispatch_1187ff96e6e.java)
> >  > >  >         at $IOEventDispatch_1187ff96e6d.connected($IOEventDispatch_1187ff96e6d.java)
> >  > >  >         at org.apache.http.impl.nio.reactor.BaseIOReactor.keyCreated(BaseIOReactor.java:182)
> >  > >  >         at org.apache.http.impl.nio.reactor.AbstractIOReactor.processNewChannels(AbstractIOReactor.java:246)
> >  > >  >         at org.apache.http.impl.nio.reactor.AbstractIOReactor.execute(AbstractIOReactor.java:153)
> >  > >  >         at org.apache.http.impl.nio.reactor.BaseIOReactor.execute(BaseIOReactor.java:70)
> >  > >  >         at org.apache.http.impl.nio.reactor.AbstractMultiworkerIOReactor$Worker.run(AbstractMultiworkerIOReactor.java:318)
> >  > >  >         at java.lang.Thread.run(Thread.java:619)
> >  > >  >
> >  > >  > Does this ring any bells? I've seen a few commits, on 1/3 and 2/3,
so
> >  > >  > i'm wondering if there's something important
> >  > >  > on those changes.
> >  > >  >
> >  > >
> >  > >  Andreas,
> >  > >
> >  > >  I believe I have fixed the problem in trunk and published the latest
> >  > >  snapshots to the repository. Could you please re-test?
> >  > >
> >  > >
> >  > >
> >  > >  Cheers,
> >  > >
> >  > >  Oleg
> >  > >
> >  > >  > TIA
> >  > >
> >  > >
> >  > >  ---------------------------------------------------------------------
> >  > >  To unsubscribe, e-mail: dev-unsubscribe@hc.apache.org
> >  > >  For additional commands, e-mail: dev-help@hc.apache.org
> >  > >
> >  > >
> >  >
> >  >
> >  >
> >  > --
> >  > Andreas Andreou - andyhot@apache.org - http://blog.andyhot.gr
> >  > Tapestry / Tacos developer
> >  > Open Source / JEE Consulting
> >  >
> >  > ---------------------------------------------------------------------
> >  > To unsubscribe, e-mail: dev-unsubscribe@hc.apache.org
> >  > For additional commands, e-mail: dev-help@hc.apache.org
> >  >
> >  >
> >
> >  ---------------------------------------------------------------------
> >  To unsubscribe, e-mail: dev-unsubscribe@hc.apache.org
> >  For additional commands, e-mail: dev-help@hc.apache.org
> >
> >
>
>
>
> --
> Andreas Andreou - andyhot@apache.org - http://blog.andyhot.gr
> Tapestry / Tacos developer
> Open Source / JEE Consulting
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@hc.apache.org
> For additional commands, e-mail: dev-help@hc.apache.org
>
>

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


Mime
View raw message