On Monday, July 30, 2012 11:41:55 PM Oleg Kalnichevski wrote: > On Mon, 2012-07-30 at 17:06 -0400, Daniel Kulp wrote: > > BTW: I think the NHttpClient.java referenced from: > > http://hc.apache.org/httpcomponents-core-ga/examples.html > > needs work as well. I don't think most of the parameters set in the > > HttpParams actually will actually go into effect. That was part of my > > original issue as the TCP_NODELAY setting didn't take effect until I set > > that on the ConnectingIOReactor instead of in the params. Not sure if > > it's a bug in the BasicNIOConnPool that was ignoring the params or a bug > > in the example. Looking at the code in BasicNIOConnPool, I think the > > connection timeout is the only setting that would work from the params. > > Higher level components such as HttpAsyncClient usually take care of > creating and initializing the I/O reactor based on the HTTP level > parameters. When using core components directly more work may be needed. > I/O reactor config in NHttpClient is obviously wrong. TCP_NODELAY should > actually be set to true per default, which is something I clearly > overlooked.
I was JUST about to ask about that. :-) false is OK for GET requests, but for PUT/POST, it completely kills performance as it adds up to a 200ms delay between sending the headers and the body as it waits for the ack of the headers. Would you like me to log some issues in JIRA for these so they get tracked? -- Daniel Kulp dk...@apache.org - http://dankulp.com/blog Talend Community Coder - http://coders.talend.com
