DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG· RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT <http://issues.apache.org/bugzilla/show_bug.cgi?id=38279>. ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND· INSERTED IN THE BUG DATABASE.
http://issues.apache.org/bugzilla/show_bug.cgi?id=38279 ------- Additional Comments From [EMAIL PROTECTED] 2006-01-29 16:51 ------- Hi Oleg, I have thought about deriving the dispatchers from AsyncHttpProcessor. But I hope we will find a way to implement an http-client package that allows for simultaneous use of synchronous and asynchronous requests. If the interceptors are registered in a base class of the dispatcher, we'll have a dispatcher and an HttpRequestExecutor to keep in sync. If the dispatcher uses a reference to the set of interceptors, we'll have only one set for both. This will require the HttpRequestExecutor to be refactored and the AsyncHttpProcessor to be derived from it. As I have mentioned before, the AsyncHttpProcessor does little more than duplicate what the HttpRequestProcessor implements, with a finer-grained API. I didn't want to change http-core at this time, so I decided to copy and paste the code initially. This situation is not meant to last, as you can see from a hidden JavaDoc comment in AsyncHttpProcessor: <!-- @@@ review to factor out duplicate code --> There are a few non-trivial things to consider when refactoring, such as the expect-continue and retry handling in HttpRequestExecutor, which needs to be implemented in a different way (or not at all) by dispatchers. But I'm confident we can come up with a patch that makes the AsyncHttpProcessor a simple stub providing access to protected methods in HttpRequestExecutor. The calls to the AsyncHttpProcessor are already encapsulated in the AbstractHttpDispatcher, so that the methods don't have to be public anymore. cheers, Roland -- Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the assignee for the bug, or are watching the assignee. --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
