Details
-
Feature Request
-
Resolution: Done
-
Major
-
None
-
None
-
0
-
0%
Description
All (Rpc,Message)Dispatcher calls which return a NotifyingFuture should have an additional parameter FutureListener, which should be registered with the future before the call is invoked.
In the current code, what can happen is:
- Caller invokes an RPC on 10 nodes
- The call is invoked
- A future is returned
- Results from the first 3 members are received and set in their futures
- The caller registers a FutureListener with the futures
- The next 7 results are returned, and the listeners invoked
--> We only get 7 results and would have to do a get on the first 3 futures to obtain all results
SOLUTION:
- Add an additional parameter (FutureListener) to the callXXWithFuture() methods, which registers the listener before invoking the call