[ 
https://issues.apache.org/jira/browse/HTTPCLIENT-1928?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16538663#comment-16538663
 ] 

silver9886 commented on HTTPCLIENT-1928:
----------------------------------------

you can log the exception in the framework ,the wrong is the complete 
method,not the connection.so i think there is no bussiness of the 
connection.the connection can be reused.even shut down the conenction,next time 
,the connection will be created next time ,and have no difference with the 
prior shut down connection. 

> HttpAsyncClient callback should ignore exception throwed by compelete method
> ----------------------------------------------------------------------------
>
>                 Key: HTTPCLIENT-1928
>                 URL: https://issues.apache.org/jira/browse/HTTPCLIENT-1928
>             Project: HttpComponents HttpClient
>          Issue Type: Improvement
>            Reporter: silver9886
>            Priority: Major
>
> If exception throwed by completed method in FutureCallback argument in 
> HttpAsyncClient.execute,the connection will be closed.
> but the exception throwed by customer code,should not affect the connection 
> managed by the framework,the connection should be connected also.
> there can be a class called FutureCallbackIgnorecompelteExcepton class.
> public abstract class FutureCallbackIgnorecompelteExcepton <T> {
> private FutureCallback<T> t;
>     void completed(T result){
>    try{
>    t.completed(result);
> }catch(Exception e)
> //ignore
> }
>     void failed(Exception ex);
>     void cancelled();
> }
> HttpAsyncClient changed as:
> HttpAsyncClient.<T> Future<T> execute(
>             AsyncRequestProducer requestProducer,
>             AsyncResponseConsumer<T> responseConsumer,
>             HttpContext context,
>             FutureCallbackIgnorecompelteExcepton <T> callback);



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to