By the way, the implementation of the async feature has been an enhncent or 
patch effort based on the legacy code. I think it definitely deserves an 
refactor. 
For example, you can find that we have three types of Result, they 
are`AsyncRpcResult`, `RpcResult`, `SimpleAsyncRpcResult`. 
* `RpcResult` are for sync call  
* `AsyncRpcResult` and `SimpleAsyncRpcResult` are for different types of async 
calls  
* `AsyncRpcResult` inside composes two futures.
But I think we should unify them to one RpcResult, which should be an instance 
of `CompletationStage`.

Given that hou have put forward many constructive suggestions recently, which 
are of great help to us. It would be great if you've got some experience or 
you'd like to take a look at this part. 

[ Full content available at: 
https://github.com/apache/incubator-dubbo/pull/3095 ]
This message was relayed via gitbox.apache.org for 
[email protected]

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

Reply via email to